Bug 129131

Summary: DefaultGCActivityCallback::doWork() should reschedule if GC is deferred
Product: WebKit Reporter: Mark Lam <mark.lam>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: fpizlo, ggaren, mhahnenberg, mmirman, msaboff, oliver
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
the patch mhahnenberg: review+

Mark Lam
Reported 2014-02-20 17:17:22 PST
Currently, DefaultGCActivityCallback::doWork() does not check if the GC needs to be deferred before commencing. Will fix. ref: <rdar://problem/15447622>
Attachments
the patch (1.70 KB, patch)
2014-02-20 18:02 PST, Mark Lam 		mhahnenberg: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2014-02-20 18:02:24 PST
Created attachment 224815 [details] the patch
Mark Hahnenberg
Comment 2 2014-02-20 18:04:43 PST
Comment on attachment 224815 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=224815&action=review r=me > Source/JavaScriptCore/ChangeLog:12 > + supposed to be deferred and re-schedule if needed. It only commences with reschedules
Mark Lam
Comment 3 2014-02-20 18:48:37 PST
Thanks. Landed in r164463: <http://trac.webkit.org/r164463>.
Note You need to log in before you can comment on or make changes to this bug.