Title: [233818] trunk/Source/WebKit
Revision
233818
Author
[email protected]
Date
2018-07-13 15:22:42 -0700 (Fri, 13 Jul 2018)

Log Message

REGRESSION (r231676): watchOS WebKit usually doesn't load in the background
https://bugs.webkit.org/show_bug.cgi?id=187663
<rdar://problem/42181185>

Reviewed by Chris Dumez.

* UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::WebPageProxy::applicationDidEnterBackground):
"Screen lock" is very aggressive on watchOS; we want to do our usual
30 seconds of loading in the background when you drop your wrist,
so disable this power optimization on that platform.

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (233817 => 233818)


--- trunk/Source/WebKit/ChangeLog	2018-07-13 21:53:14 UTC (rev 233817)
+++ trunk/Source/WebKit/ChangeLog	2018-07-13 22:22:42 UTC (rev 233818)
@@ -1,3 +1,17 @@
+2018-07-13  Tim Horton  <[email protected]>
+
+        REGRESSION (r231676): watchOS WebKit usually doesn't load in the background
+        https://bugs.webkit.org/show_bug.cgi?id=187663
+        <rdar://problem/42181185>
+
+        Reviewed by Chris Dumez.
+
+        * UIProcess/ios/WebPageProxyIOS.mm:
+        (WebKit::WebPageProxy::applicationDidEnterBackground):
+        "Screen lock" is very aggressive on watchOS; we want to do our usual
+        30 seconds of loading in the background when you drop your wrist,
+        so disable this power optimization on that platform.
+
 2018-07-13  Chris Dumez  <[email protected]>
 
         WebResourceLoader may try to send a IPC with a destination ID that is 0

Modified: trunk/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm (233817 => 233818)


--- trunk/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm	2018-07-13 21:53:14 UTC (rev 233817)
+++ trunk/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm	2018-07-13 22:22:42 UTC (rev 233818)
@@ -644,10 +644,12 @@
 {
     bool isSuspendedUnderLock = [UIApp isSuspendedUnderLock];
 
+#if !PLATFORM(WATCHOS)
     // We normally delay process suspension when the app is backgrounded until the current page load completes. However,
     // we do not want to do so when the screen is locked for power reasons.
     if (isSuspendedUnderLock)
         NavigationState::fromWebPage(*this).releaseNetworkActivityToken(NavigationState::NetworkActivityTokenReleaseReason::ScreenLocked);
+#endif
     m_process->send(Messages::WebPage::ApplicationDidEnterBackground(isSuspendedUnderLock), m_pageID);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to