[webkit-reviews] review granted: [Bug 131860] [iOS] REGRESSION (r167039): WebKit incorrectly sets a background process assertion state when displaying a PDF : [Attachment 229678] Patch

[bugzilla-daemon at webkit.org]

Tim Horton <thorton at apple.com> has granted Andy Estes <aestes at apple.com>'s
request for review:
Bug 131860: [iOS] REGRESSION (r167039): WebKit incorrectly sets a background
process assertion state when displaying a PDF
https://bugs.webkit.org/show_bug.cgi?id=131860

Attachment 229678: Patch
https://bugs.webkit.org/attachment.cgi?id=229678&action=review

------- Additional Comments from Tim Horton <thorton at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=229678&action=review


> Source/WebKit2/UIProcess/ios/PageClientImplIOS.mm:130
> -    return [m_contentView window];
> +    return [m_webView window];

We also need to notify the view state that it needs to recompute when WKWebView
goes into/out of window, just like WKContentView does:
WKWebView

- (void)didMoveToWindow
{
    _page->viewStateDidChange(ViewState::IsInWindow);
}