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

Issue 1677033002: Pass executionContext to document.createEvent() (Closed)

Created:
4 years, 10 months ago by davve
Modified:
4 years, 10 months ago
Reviewers:
philipj_slow
CC:
chromium-reviews, sof, eae+blinkwatch, blink-reviews-dom_chromium.org, dglazkov+blink, blink-reviews, rwlbuis
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Pass executionContext to document.createEvent() In the long run, createEvent() should only work for whitelisted event interfaces. An execution context is needed for taking the first steps by adding use counters and doing deprecations. BUG=569690 Committed: https://crrev.com/c9db86b71f614522e2174e564f85dfcf4a32864e Cr-Commit-Position: refs/heads/master@{#374913}

Patch Set 1 #

Messages

Total messages: 15 (7 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1677033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1677033002/1
4 years, 10 months ago (2016-02-08 14:28:45 UTC) #2
davve
To be judged together with dependent CL.
4 years, 10 months ago (2016-02-08 16:13:22 UTC) #5
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/170638)
4 years, 10 months ago (2016-02-08 16:31:07 UTC) #7
philipj_slow
lgtm
4 years, 10 months ago (2016-02-08 17:21:40 UTC) #8
philipj_slow
On 2016/02/08 17:21:40, philipj_UTC7 wrote: > lgtm No CQ in case there's something to iterate ...
4 years, 10 months ago (2016-02-08 17:22:05 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1677033002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1677033002/1
4 years, 10 months ago (2016-02-11 15:31:38 UTC) #11
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 10 months ago (2016-02-11 16:52:52 UTC) #13
commit-bot: I haz the power
4 years, 10 months ago (2016-02-16 22:35:59 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/c9db86b71f614522e2174e564f85dfcf4a32864e
Cr-Commit-Position: refs/heads/master@{#374913}

Powered by Google App Engine
This is Rietveld 408576698