Title: [242554] trunk/Source/WebKit
Revision
242554
Author
[email protected]
Date
2019-03-06 11:10:06 -0800 (Wed, 06 Mar 2019)

Log Message

REGRESSION (r238490): YouTube.com: Returning PiP to Safari after sleeping device loses page
https://bugs.webkit.org/show_bug.cgi?id=195364
<rdar://problem/48538837>

Reviewed by Geoffrey Garen.

Make sure we unset the LayerTreeFreezeReason::ProcessSuspended layer tree freeze reason
when WebPage::applicationWillEnterForeground() is called. This restores pre-r238490
behavior and addresses the issue with PiP on youtube.com.

* WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::applicationWillEnterForeground):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (242553 => 242554)


--- trunk/Source/WebKit/ChangeLog	2019-03-06 19:08:28 UTC (rev 242553)
+++ trunk/Source/WebKit/ChangeLog	2019-03-06 19:10:06 UTC (rev 242554)
@@ -1,3 +1,18 @@
+2019-03-06  Chris Dumez  <[email protected]>
+
+        REGRESSION (r238490): YouTube.com: Returning PiP to Safari after sleeping device loses page
+        https://bugs.webkit.org/show_bug.cgi?id=195364
+        <rdar://problem/48538837>
+
+        Reviewed by Geoffrey Garen.
+
+        Make sure we unset the LayerTreeFreezeReason::ProcessSuspended layer tree freeze reason
+        when WebPage::applicationWillEnterForeground() is called. This restores pre-r238490
+        behavior and addresses the issue with PiP on youtube.com.
+
+        * WebProcess/WebPage/ios/WebPageIOS.mm:
+        (WebKit::WebPage::applicationWillEnterForeground):
+
 2019-03-05  Dean Jackson  <[email protected]>
 
         Fix WKPasswordView compilation on iOS

Modified: trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm (242553 => 242554)


--- trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm	2019-03-06 19:08:28 UTC (rev 242553)
+++ trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm	2019-03-06 19:10:06 UTC (rev 242554)
@@ -3002,6 +3002,9 @@
 {
     m_isSuspendedUnderLock = false;
     cancelMarkLayersVolatile();
+
+    // FIXME: In iOS, we sometimes never unset ProcessSuspended. See <rdar://problem/48538837>.
+    unfreezeLayerTree(LayerTreeFreezeReason::ProcessSuspended);
     unfreezeLayerTree(LayerTreeFreezeReason::BackgroundApplication);
 
     [[NSNotificationCenter defaultCenter] postNotificationName:WebUIApplicationWillEnterForegroundNotification object:nil userInfo:@{@"isSuspendedUnderLock": @(isSuspendedUnderLock)}];
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to