[Webkit-unassigned] [Bug 126214] New: Merge PageVisibilityState & ViewState::IsVisible in WebKit2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Dec 24 08:36:52 PST 2013


https://bugs.webkit.org/show_bug.cgi?id=126214

           Summary: Merge PageVisibilityState & ViewState::IsVisible in
                    WebKit2
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit2
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: barraclough at apple.com


WebKit2 redundantly tracks the visibility of the view through two mechanisms - the visibility state, and the view state. Remove visibility state from the WebKit2 layer. The visibility state also tracks the 'pretender' state - so split this out and handle it separately (a change we should make in WebCore, too).

Removing the redundancy also removes the ability from the API to set a fake visibility state (IsVisible tracks the actual visibility of the view). Through private API (WKPageSetVisibilityState) a client could previously request the view be reported as hidden/visible, but this didn't really work - the override was not enforced and the API may reset the state at an arbitrary point. The mechanism is only used by testing code, which instead should actually update the view visibility (this tests more of the actual visibility mechanisms used by the browser). The one aspect of the API relied on by existing clients is the ability to initialize a hidden view as prerender -continue to support this specific functionality via WKPageSetVisibilityState, to maintain backwards compatibility.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list