Bug 84094

Summary: [MutationObservers] enable V8RecursionScope debug ASSERT()
Product: WebKit Reporter: Rafael Weinstein <rafaelw>
Component: DOMAssignee: Rafael Weinstein <rafaelw>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, adamk, fishd, ojan, rniwa, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 68729    
Attachments:
Description Flags
Patch none

Rafael Weinstein
Reported 2012-04-16 16:20:39 PDT
This will cause a failure in chromium debug tests if any new usage of script invocation to a context initialized in WebKit.cpp is added without either going through V8Proxy or whitelisting the call as "internal" by allocating a V8RecusionScope::MicrotaskSuppression
Attachments
Patch (1.43 KB, patch)
2012-04-16 16:26 PDT, Rafael Weinstein 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Rafael Weinstein
Comment 1 2012-04-16 16:26:44 PDT
Created attachment 137424 [details] Patch
Rafael Weinstein
Comment 2 2012-04-16 16:28:14 PDT
This will wait on (at least) https://chromiumcodereview.appspot.com/9969191/
Rafael Weinstein
Comment 3 2012-04-17 15:45:48 PDT
Chromium patch has landed and try job for this patch has cleared. I'd like to land this now.
Ryosuke Niwa
Comment 4 2012-04-17 15:49:38 PDT
Great change!
WebKit Review Bot
Comment 5 2012-04-17 16:03:02 PDT
Comment on attachment 137424 [details] Patch Clearing flags on attachment: 137424 Committed r114458: <http://trac.webkit.org/changeset/114458>
WebKit Review Bot
Comment 6 2012-04-17 16:03:15 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.