Bug 142595

Summary: Users of Heap::deprecatedReportExtraMemory should switch to reportExtraMemoryAllocated+reportExtraMemoryVisited
Product: WebKit Reporter: Geoffrey Garen <ggaren>
Component: WebKit Misc.Assignee: Geoffrey Garen <ggaren>
Status: REOPENED    
Severity: Normal CC: cdumez, fpizlo, kling, mhahnenb
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=158274
Bug Depends on:    
Bug Blocks: 142457    
Attachments:
Description Flags
Patch kling: review+

Geoffrey Garen
Reported 2015-03-11 14:32:26 PDT
This will reduce GC pressure when GenGC is successful.
Attachments
Patch (3.23 KB, patch)
2015-03-11 17:03 PDT, Geoffrey Garen 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Geoffrey Garen
Comment 1 2015-03-11 17:03:17 PDT
Created attachment 248465 [details] Patch
Andreas Kling
Comment 2 2015-03-11 17:14:08 PDT
Comment on attachment 248465 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=248465&action=review r=me > Source/WebCore/html/HTMLCanvasElement.h:138 > + size_t memoryCost() const; memoryCost() and not extraMemoryCost() seems like a mild misnomer, but it's not terribly important.
Geoffrey Garen
Comment 3 2015-03-11 17:15:48 PDT
Committed r181415: <http://trac.webkit.org/changeset/181415>
Geoffrey Garen
Comment 4 2015-03-11 17:16:15 PDT
Keeping this open to cover the non-canvas classes.
Note You need to log in before you can comment on or make changes to this bug.