Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(642)

Issue 187313004: Merge 166999 "Add cross-origin BindingsSecurity checks to 'Event..." (Closed)

Created:
6 years, 9 months ago by Mike West
Modified:
6 years, 9 months ago
CC:
blink-reviews, Nils Barth (inactive), kojih, arv+blink, jsbell+bindings_chromium.org, sof, abarth-chromium, marja+watch_chromium.org, adamk+blink_chromium.org, haraken, Nate Chapin, watchdog-blink-watchlist_google.com, Inactive
Visibility:
Public.

Description

Merge 166999 "Add cross-origin BindingsSecurity checks to 'Event..." > Add cross-origin BindingsSecurity checks to 'EventTarget::dispatchEvent'. > > BUG=342618 > > Review URL: https://codereview.chromium.org/150203016 TBR=mkwst@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=168445

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -3 lines) Patch
A + LayoutTests/http/tests/security/cross-frame-access-dispatchEvent.html View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/http/tests/security/cross-frame-access-dispatchEvent-expected.txt View 0 chunks +-1 lines, --1 lines 0 comments Download
M Source/bindings/scripts/code_generator_v8.pm View 3 chunks +10 lines, -5 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mike West
6 years, 9 months ago (2014-03-05 07:52:37 UTC) #1
Mike West
Committed patchset #1 manually as r168445.
6 years, 9 months ago (2014-03-05 07:53:54 UTC) #2
Nils Barth (inactive)
6 years, 9 months ago (2014-03-05 08:02:37 UTC) #3
Message was sent while issue was closed.
Hi Mike,
might I ask what this CL is doing?
It looks mostly empty, and it's modifying code_generator_v8.pm,
which is gone.

Powered by Google App Engine
This is Rietveld 408576698