Apps that use both WK1 and WK2 can crash creating a WKWebsiteDataStore.
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Dec 2017 00:14:24 +0000 (00:14 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Dec 2017 00:14:24 +0000 (00:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=180953

Reviewed by Chris Dumez.

* UIProcess/WebsiteData/WebsiteDataStore.cpp: Make an isMainThread() asset be an isUIThread() assert.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@226083 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp

index afbb2e2..a692ca9 100644 (file)
@@ -1,3 +1,12 @@
+2017-12-18  Brady Eidson  <beidson@apple.com>
+
+        Apps that use both WK1 and WK2 can crash creating a WKWebsiteDataStore.
+        https://bugs.webkit.org/show_bug.cgi?id=180953
+
+        Reviewed by Chris Dumez.
+
+        * UIProcess/WebsiteData/WebsiteDataStore.cpp: Make an isMainThread() asset be an isUIThread() assert.
+
 2017-12-18  Brent Fulgham  <bfulgham@apple.com>
 
         [macOS, iOS] Add new mach lookup port for CoreMedia remote control
index 7d3afe3..deab0bc 100644 (file)
@@ -57,7 +57,7 @@ namespace WebKit {
 
 static HashMap<PAL::SessionID, WebsiteDataStore*>& nonDefaultDataStores()
 {
-    RELEASE_ASSERT(isMainThread());
+    RELEASE_ASSERT(isUIThread());
     static NeverDestroyed<HashMap<PAL::SessionID, WebsiteDataStore*>> map;
     return map;
 }