Bug 163821

Summary: WebPageProxy should not need PageActivityState
Product: WebKit Reporter: Gavin Barraclough <barraclough>
Component: WebKit2Assignee: Gavin Barraclough <barraclough>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fix ggaren: review+

Gavin Barraclough
Reported 2016-10-21 17:12:20 PDT
The PageActivityState is currently plumbed back from WebCore up to the UI process, to determine whether to enabled process suppression. However the information it contains (whether a page load is ongoing, whether audio is playing) is already available via other means. Remove this use of PageActivityState.
Attachments
Fix (13.64 KB, patch)
2016-10-21 17:18 PDT, Gavin Barraclough 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2016-10-21 17:18:50 PDT
Created attachment 292436 [details] Fix
Geoffrey Garen
Comment 2 2016-10-21 17:27:16 PDT
Comment on attachment 292436 [details] Fix r=me It scares me a little how distributed this stuff is throughout the codebase.
Gavin Barraclough
Comment 3 2016-10-21 23:10:17 PDT
Committed revision 207706.
Note You need to log in before you can comment on or make changes to this bug.