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