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

Description Chris Dumez 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.
Comment 1 Radar WebKit Bug Importer 2017-03-09 12:29:02 PST 
<rdar://problem/30954003>
Comment 2 Alexey Proskuryakov 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.
Comment 3 Chris Dumez 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.
Comment 4 Chris Dumez 2017-03-10 10:05:56 PST 
Created attachment 304050 [details]
Patch
Comment 5 Andreas Kling 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.
Comment 6 WebKit Commit Bot 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>
Comment 7 WebKit Commit Bot 2017-03-13 04:16:39 PDT 
All reviewed patches have been landed.  Closing bug.