[Webkit-unassigned] [Bug 54181] Implement Page Visibility API.

[bugzilla-daemon at webkit.org]

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





--- Comment #29 from Shishir Agrawal <shishir at chromium.org>  2011-05-05 16:18:49 PST ---
(From update of attachment 92449)
View in context: https://bugs.webkit.org/attachment.cgi?id=92449&action=review

>> Source/WebKit/chromium/src/WebViewImpl.cpp:2573
>> +    switch (visibilityState) {
> 
> It seems like this 10-line block could be written as:
>   ASSERT(visibilityState == WebPageVisibilityStateVisible || visibilityState == WebPageVisibilityStateHidden);
>   m_page->setVisibilityState(visibilityState, isInitialState);

Not really, since the two enums are different. The input is WebPageVisibilityState and the other one is PageVisibilityState.

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