Diff
Modified: trunk/Source/WebKit/ChangeLog (245326 => 245327)
--- trunk/Source/WebKit/ChangeLog 2019-05-15 15:59:21 UTC (rev 245326)
+++ trunk/Source/WebKit/ChangeLog 2019-05-15 16:35:58 UTC (rev 245327)
@@ -1,3 +1,21 @@
+2019-05-15 Youenn Fablet <[email protected]>
+
+ Constant crashes under WebPage::isThrottleable() after r245299
+ https://bugs.webkit.org/show_bug.cgi?id=197902
+ <rdar://problem/50793796>
+
+ Reviewed by Alex Christensen.
+
+ Update throttle state after a run loop iteration when page state changes to make sure the pageMap does not have null pages.
+ Add appNapEnabled preference that is used by added API test.
+
+ * WebProcess/WebPage/WebPage.cpp:
+ (WebKit::WebPage::updateThrottleState):
+ * UIProcess/API/Cocoa/WKPreferences.mm:
+ (-[WKPreferences _setAppNapEnabled:]):
+ (-[WKPreferences _appNapEnabled]):
+ * UIProcess/API/Cocoa/WKPreferencesPrivate.h:
+
2019-05-15 Alex Christensen <[email protected]>
Allow NSFileCoordinator to be called from WebContent process
Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm (245326 => 245327)
--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm 2019-05-15 15:59:21 UTC (rev 245326)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferences.mm 2019-05-15 16:35:58 UTC (rev 245327)
@@ -1265,6 +1265,16 @@
return _preferences->aggressiveTileRetentionEnabled();
}
+- (void)_setAppNapEnabled:(BOOL)enabled
+{
+ _preferences->setPageVisibilityBasedProcessSuppressionEnabled(enabled);
+}
+
+- (BOOL)_appNapEnabled
+{
+ return _preferences->pageVisibilityBasedProcessSuppressionEnabled();
+}
+
#endif // PLATFORM(MAC)
- (BOOL)_javaScriptCanAccessClipboard
Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h (245326 => 245327)
--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h 2019-05-15 15:59:21 UTC (rev 245326)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKPreferencesPrivate.h 2019-05-15 16:35:58 UTC (rev 245327)
@@ -200,6 +200,7 @@
@property (nonatomic, setter=_setWantsBalancedSetDefersLoadingBehavior:) BOOL _wantsBalancedSetDefersLoadingBehavior WK_API_AVAILABLE(macos(10.14));
@property (nonatomic, setter=_setWebAudioEnabled:) BOOL _webAudioEnabled WK_API_AVAILABLE(macos(10.14));
@property (nonatomic, setter=_setAggressiveTileRetentionEnabled:) BOOL _aggressiveTileRetentionEnabled WK_API_AVAILABLE(macos(10.14));
+@property (nonatomic, setter=_setAppNapEnabled:) BOOL _appNapEnabled WK_API_AVAILABLE(macos(WK_MAC_TBA));
#endif
@end
Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp (245326 => 245327)
--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp 2019-05-15 15:59:21 UTC (rev 245326)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp 2019-05-15 16:35:58 UTC (rev 245327)
@@ -776,10 +776,12 @@
m_userActivity.start();
#if ENABLE(SERVICE_WORKER)
- if (auto* connection = ServiceWorkerProvider::singleton().existingServiceWorkerConnectionForSession(sessionID())) {
- if (isThrottleable != connection->isThrottleable())
- connection->updateThrottleState();
- }
+ RunLoop::main().dispatch([isThrottleable, sessionID = sessionID()] {
+ if (auto* connection = ServiceWorkerProvider::singleton().existingServiceWorkerConnectionForSession(sessionID)) {
+ if (isThrottleable != connection->isThrottleable())
+ connection->updateThrottleState();
+ }
+ });
#endif
}
Modified: trunk/Tools/ChangeLog (245326 => 245327)
--- trunk/Tools/ChangeLog 2019-05-15 15:59:21 UTC (rev 245326)
+++ trunk/Tools/ChangeLog 2019-05-15 16:35:58 UTC (rev 245327)
@@ -1,3 +1,13 @@
+2019-05-15 Youenn Fablet <[email protected]>
+
+ Constant crashes under WebPage::isThrottleable() after r245299
+ https://bugs.webkit.org/show_bug.cgi?id=197902
+ <rdar://problem/50793796>
+
+ Reviewed by Alex Christensen.
+
+ * TestWebKitAPI/Tests/WebKitCocoa/ServiceWorkerBasic.mm:
+
2019-05-15 Philippe Normand <[email protected]>
[GTK][jhbuild] misc cleanups and bumps
Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/ServiceWorkerBasic.mm (245326 => 245327)
--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/ServiceWorkerBasic.mm 2019-05-15 15:59:21 UTC (rev 245326)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/ServiceWorkerBasic.mm 2019-05-15 16:35:58 UTC (rev 245327)
@@ -27,6 +27,8 @@
#import "PlatformUtilities.h"
#import "Test.h"
+#import "TestNavigationDelegate.h"
+#import "TestWKWebView.h"
#import <WebKit/WKPreferencesPrivate.h>
#import <WebKit/WKProcessPoolPrivate.h>
#import <WebKit/WKURLSchemeHandler.h>
@@ -52,6 +54,7 @@
};
static bool done;
+static bool didFinishNavigation;
static String expectedMessage;
static String retrievedString;
@@ -1619,6 +1622,74 @@
EXPECT_EQ(0U, processPool._serviceWorkerProcessCount);
}
+TEST(ServiceWorkers, ThrottleCrash)
+{
+ ASSERT(mainBytes);
+ ASSERT(scriptBytes);
+
+ [WKWebsiteDataStore _allowWebsiteDataRecordsForAllOrigins];
+
+ // Start with a clean slate data store
+ [[WKWebsiteDataStore defaultDataStore] removeDataOfTypes:[WKWebsiteDataStore allWebsiteDataTypes] modifiedSince:[NSDate distantPast] completionHandler:^() {
+ done = true;
+ }];
+ TestWebKitAPI::Util::run(&done);
+ done = false;
+
+ auto messageHandler = adoptNS([[SWMessageHandler alloc] init]);
+
+ auto handler = adoptNS([[SWSchemes alloc] init]);
+ handler->resources.set("sw1://host/main.html", ResourceInfo { @"text/html", mainBytes });
+ handler->resources.set("sw1://host/sw.js", ResourceInfo { @"application/_javascript_", scriptBytes });
+
+ auto navigationDelegate = adoptNS([[TestNavigationDelegate alloc] init]);
+ [navigationDelegate setDidFinishNavigation:^(WKWebView *, WKNavigation *) {
+ didFinishNavigation = true;
+ }];
+
+ auto configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
+#if PLATFORM(MAC)
+ [[configuration preferences] _setAppNapEnabled:YES];
+#endif
+ [[configuration userContentController] addScriptMessageHandler:messageHandler.get() name:@"sw"];
+ [configuration setURLSchemeHandler:handler.get() forURLScheme:@"sw1"];
+
+ auto *processPool = configuration.get().processPool;
+ [processPool _registerURLSchemeServiceWorkersCanHandle:@"sw1"];
+
+ auto webView1 = adoptNS([[TestWKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:configuration.get() addToWindow: YES]);
+ [webView1 setNavigationDelegate:navigationDelegate.get()];
+
+ // Let's make it so that webView1 be app nappable after loading is completed.
+ [webView1.get().window resignKeyWindow];
+#if PLATFORM(MAC)
+ [webView1.get().window orderOut:nil];
+#endif
+
+ auto *request1 = [NSURLRequest requestWithURL:[NSURL URLWithString:@"sw1://host/main.html"]];
+ [webView1 loadRequest:request1];
+
+ didFinishNavigation = false;
+ TestWebKitAPI::Util::run(&didFinishNavigation);
+
+ auto webView2Configuration = adoptNS([[WKWebViewConfiguration alloc] init]);
+#if PLATFORM(MAC)
+ [[webView2Configuration preferences] _setAppNapEnabled:NO];
+#endif
+ [webView2Configuration setProcessPool:processPool];
+ [[webView2Configuration userContentController] addScriptMessageHandler:messageHandler.get() name:@"sw"];
+ [webView2Configuration setURLSchemeHandler:handler.get() forURLScheme:@"sw1"];
+ webView2Configuration.get()._relatedWebView = webView1.get();
+
+ auto webView2 = adoptNS([[WKWebView alloc] initWithFrame:NSMakeRect(0, 0, 800, 600) configuration:webView2Configuration.get()]);
+ [webView2 setNavigationDelegate:navigationDelegate.get()];
+
+ [webView2 loadRequest:request1];
+
+ didFinishNavigation = false;
+ TestWebKitAPI::Util::run(&didFinishNavigation);
+}
+
TEST(ServiceWorkers, LoadData)
{
ASSERT(mainBytes);
