Bug 153978

Summary: Visiting a WeakBlock should report bytes visited, since we reported them allocated.
Product: WebKit Reporter: Andreas Kling <kling>
Component: JavaScriptCoreAssignee: Andreas Kling <kling>
Status: REOPENED    
Severity: Normal CC: cdumez, commit-queue, ggaren, keith_miller, mark.lam, msaboff, saam
Priority: P2 Keywords: Performance
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 154078    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Andreas Kling
Reported 2016-02-08 01:44:13 PST
WeakBlock should have both allocation + visitation memory accounting.
Attachments
Patch (2.14 KB, patch)
2016-02-08 01:45 PST, Andreas Kling 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2016-02-08 01:45:03 PST
Created attachment 270849 [details] Patch
WebKit Commit Bot
Comment 2 2016-02-08 08:25:26 PST
Comment on attachment 270849 [details] Patch Clearing flags on attachment: 270849 Committed r196251: <http://trac.webkit.org/changeset/196251>
WebKit Commit Bot
Comment 3 2016-02-08 08:25:31 PST
All reviewed patches have been landed. Closing bug.
Chris Dumez
Comment 4 2016-02-08 14:59:58 PST
This may have caused a Dromaeo ∋ JavaScript Library Tests ~8% regression on MacBook Air.
WebKit Commit Bot
Comment 5 2016-02-10 10:07:58 PST
Re-opened since this is blocked by bug 154078
Note You need to log in before you can comment on or make changes to this bug.