Bug 159597

Summary: Perf dashboard can consume 50-70% of CPU on MacBook even if user is not interacting at all
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Perf DashboardAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, cdumez, commit-queue, dewei_zhu, kling, koivisto, rniwa, slewis
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fixes the bug
none
Use Set for render queue none

Ryosuke Niwa
Reported 2016-07-08 23:31:21 PDT
Apparently continuously polling on requestAnimationFrame isn't great for battery life :(
Attachments
Fixes the bug (20.62 KB, patch)
2016-07-08 23:49 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Use Set for render queue (20.62 KB, patch)
2016-07-08 23:53 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2016-07-08 23:49:33 PDT
Created attachment 283247 [details] Fixes the bug
Ryosuke Niwa
Comment 2 2016-07-08 23:53:34 PDT
Created attachment 283248 [details] Use Set for render queue
Chris Dumez
Comment 3 2016-07-09 08:17:54 PDT
Comment on attachment 283248 [details] Use Set for render queue rs=me
WebKit Commit Bot
Comment 4 2016-07-09 15:59:36 PDT
Comment on attachment 283248 [details] Use Set for render queue Clearing flags on attachment: 283248 Committed r203035: <http://trac.webkit.org/changeset/203035>
WebKit Commit Bot
Comment 5 2016-07-09 15:59:43 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.