Bug 130531

Summary: Add some assertions back
Product: WebKit Reporter: Mark Hahnenberg <mhahnenberg>
Component: JavaScriptCoreAssignee: Mark Hahnenberg <mhahnenberg>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Mark Hahnenberg
Reported 2014-03-20 13:03:42 PDT
We removed a useful set of assertions for verifying that MarkedBlocks were in the state that we expected them to be in after clearing marks in the Heap. We should add these back to catch bugs earlier.
Attachments
Patch (2.51 KB, patch)
2014-03-20 13:05 PDT, Mark Hahnenberg 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2014-03-20 13:05:17 PDT
Created attachment 227322 [details] Patch
Geoffrey Garen
Comment 2 2014-03-20 15:52:50 PDT
Comment on attachment 227322 [details] Patch r=me
WebKit Commit Bot
Comment 3 2014-03-20 17:11:22 PDT
Comment on attachment 227322 [details] Patch Clearing flags on attachment: 227322 Committed r166028: <http://trac.webkit.org/changeset/166028>
WebKit Commit Bot
Comment 4 2014-03-20 17:11:24 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.