Home | Search | chromium - Builders
Login

Builder GPU FYI Mac Builder Build 23956 Apple OSX

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)

Timing:

Create Friday, 12-Oct-18 09:48:11 UTC
Start Friday, 12-Oct-18 09:48:13 UTC
End Friday, 12-Oct-18 09:54:17 UTC
Pending 2 secs
Execution 6 mins 4 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