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+

Description Gavin Barraclough 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.
Comment 1 Gavin Barraclough 2016-10-21 17:18:50 PDT
Created attachment 292436 [details]
Fix
Comment 2 Geoffrey Garen 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.
Comment 3 Gavin Barraclough 2016-10-21 23:10:17 PDT
Committed revision 207706.