Home | Search | chromium - Builders
Login

Builder win-rel Build 4272 Microsoft Windows

Results:

Success

Trigger Info:

Projectchromium
Revision1d2509fc2ebc7c9848726c1b208483e1e1efc485
Got Revision1d2509fc2ebc7c9848726c1b208483e1e1efc485

Execution:

Steps and Logfiles:

Show:
  1. ( 106751 days 23 hrs ) Failed to fetch step information from LogDog
    Log stream has no annotation entries

Build Properties:

NameValueSource

Blamelist:

  1. Yuki Yamada (yukiyohnoyoudont@google.com)
  2. Gauthier Ambard (gambardohnoyoudont@chromium.org)

Timing:

Create Friday, 12-Oct-18 09:49:48 UTC
Start Friday, 12-Oct-18 09:49:49 UTC
End Friday, 12-Oct-18 10:06:37 UTC
Pending 1 secs
Execution 16 mins 48 secs

All Changes:

  1. Added security check for cross origin

    Changed by Yuki Yamada - yukiyohnoyoudont@google.com
    Changed at Friday, 12-Oct-18 09:45:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 1d2509fc2ebc7c9848726c1b208483e1e1efc485

    Comments

    Added security check for cross origin
    
    This CL adds a security check for cross origin with
    BindingSecurity::ShouldAllowAccessToCreationContext().
    |js_event|, a V8 wrapper object for event object, must be created in the
    relevant realm of the event target, but it is possible that listener's
    relevant context cannnot access the relevant realm of event target
    (ex. when Document.origin is changed).
    We have to check this before invoking event listener.
    
    Bug: 872138, 884516
    Change-Id: Ic5d0c8e6cda4db57a2097ce230e75cc59905b350
    Reviewed-on: https://chromium-review.googlesource.com/c/1270300
    Commit-Queue: Yuki Yamada <yukiy@google.com>
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
    Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599154}

    Changed files

    • third_party/blink/renderer/bindings/core/v8/js_based_event_listener.cc
  2. [iOS] Support Dynamic Type for ReadingList empty string

    Changed by Gauthier Ambard - gambardohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 09:41:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision fe100b299e7bc889ccafc03cf8854efe64020679

    Comments

    [iOS] Support Dynamic Type for ReadingList empty string
    
    This CL changes the string displayed when the ReadingList is empty so
    it doesn't overflow below the bottom and top toolbar when the font is
    too big. It does that by limiting the font size.
    
    Bug: 893535
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
    Change-Id: I03335f2c1fee192a765b6e85011627c0f5ae2019
    Reviewed-on: https://chromium-review.googlesource.com/c/1276925
    Commit-Queue: Gauthier Ambard <gambard@chromium.org>
    Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599153}

    Changed files

    • ios/chrome/browser/ui/reading_list/empty_reading_list_message_util.mm
    • ios/chrome/browser/ui/reading_list/reading_list_table_view_controller.mm