Bug 169425

Summary: [WK2] Only report background WebProcesses as unresponsive in the background after 90 seconds
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, ap, barraclough, benjamin, cmarcelo, commit-queue, dbates, ggaren, kling, koivisto, mitz, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 169456    
Attachments:
Description Flags
Patch none

Chris Dumez
Reported 2017-03-09 12:28:30 PST
Only report background WebProcesses as unresponsive in the background after 90 seconds, instead of the usual 3 seconds for foreground ones.
Attachments
Patch (24.08 KB, patch)
2017-03-10 10:05 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2017-03-09 12:29:02 PST
<rdar://problem/30954003>
Alexey Proskuryakov
Comment 2 2017-03-09 14:36:51 PST
When does this kick in? I would expect background processes to never be marked as unresponsive, as they don't get events.
Chris Dumez
Comment 3 2017-03-09 14:38:12 PST
(In reply to comment #2) > When does this kick in? I would expect background processes to never be > marked as unresponsive, as they don't get events. This is a follow-up to: http://trac.webkit.org/r212832 http://trac.webkit.org/r212619 We do mark background processes as unresponsive now.
Chris Dumez
Comment 4 2017-03-10 10:05:56 PST
Created attachment 304050 [details] Patch
Andreas Kling
Comment 5 2017-03-13 03:48:12 PDT
Comment on attachment 304050 [details] Patch r=me This makes me wonder if 30 seconds isn't too aggressive for checking memory usage in background tabs, too.
WebKit Commit Bot
Comment 6 2017-03-13 04:16:33 PDT
Comment on attachment 304050 [details] Patch Clearing flags on attachment: 304050 Committed r213824: <http://trac.webkit.org/changeset/213824>
WebKit Commit Bot
Comment 7 2017-03-13 04:16:39 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.