Bug 170460

Summary: Tweak how long we allow a process to go over its background CPU limit
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, commit-queue, ggaren, kling, koivisto, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2017-04-04 11:18:42 PDT
Tweak how long we allow a process to go over its background CPU limit. We currently use a conservative 15 minutes but we should probably use something lower to prevent unnecessary battery drain.
Attachments
Patch (1.46 KB, patch)
2017-05-01 08:42 PDT, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2017-04-04 11:44:27 PDT
<rdar://problem/31433141>
Chris Dumez
Comment 2 2017-05-01 08:42:20 PDT
Created attachment 308721 [details] Patch
Geoffrey Garen
Comment 3 2017-05-01 10:38:49 PDT
Comment on attachment 308721 [details] Patch r=me What methodology are we going to use to evaluate these limits? Do we keeps statistics on aggregate kills, or on aggregate battery usage in the field?
WebKit Commit Bot
Comment 4 2017-05-01 11:09:55 PDT
Comment on attachment 308721 [details] Patch Clearing flags on attachment: 308721 Committed r216020: <http://trac.webkit.org/changeset/216020>
WebKit Commit Bot
Comment 5 2017-05-01 11:09:56 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.