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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 193123002: Use new V8 Microtask Queue API (Closed)

Created:
6 years, 9 months ago by adamk
Modified:
6 years, 9 months ago
Reviewers:
rafaelw
CC:
blink-reviews, eae+blinkwatch, apavlov+blink_chromium.org, adamk+blink_chromium.org, aandrey+blink_chromium.org, rwlbuis, Nils Barth (inactive), caseq+blink_chromium.org, kojih, arv+blink, yurys+blink_chromium.org, abarth-chromium, marja+watch_chromium.org, dglazkov+blink, devtools-reviews_chromium.org, loislo+blink_chromium.org, sof, lushnikov+blink_chromium.org, eustas+blink_chromium.org, paulirish+reviews_chromium.org, haraken, Nate Chapin, jsbell+bindings_chromium.org, alph+blink_chromium.org, kouhei+bindings_chromium.org, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, Inactive
Visibility:
Public.

Description

Use new V8 Microtask Queue API V8 now hosts the end-of-microtask work queue. This allows consistent scheduling of microtask work regardless of where each task happens to be implemented (e.g., Object.observe callbacks are handled by V8 while MutationObserver callbacks are handled by Blink). This is a re-upload of Rafael Weinstein's patch: https://codereview.chromium.org/167683003/ TBR=rafaelw@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -25 lines) Patch
M LayoutTests/inspector/sources/debugger/async-callstack-mutation-observer-expected.txt View 6 chunks +10 lines, -0 lines 0 comments Download
M Source/bindings/v8/ScriptRegexp.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/bindings/v8/V8PerIsolateData.h View 2 chunks +2 lines, -2 lines 0 comments Download
M Source/bindings/v8/V8PerIsolateData.cpp View 1 chunk +4 lines, -4 lines 0 comments Download
M Source/core/dom/Microtask.cpp View 1 chunk +26 lines, -17 lines 0 comments Download
M Source/web/WebKit.cpp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
adamk
The CQ bit was checked by adamk@chromium.org
6 years, 9 months ago (2014-03-10 18:59:44 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/adamk@chromium.org/193123002/1
6 years, 9 months ago (2014-03-10 18:59:49 UTC) #2
adamk
6 years, 9 months ago (2014-03-10 18:59:57 UTC) #3
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-10 21:01:02 UTC) #4
commit-bot: I haz the power
6 years, 9 months ago (2014-03-10 21:01:02 UTC) #5 
Try jobs failed on following builders: linux_blink

Powered by Google App Engine
This is Rietveld 408576698