Changeset 226083 in webkit

Timestamp:

Dec 18, 2017 4:14:24 PM (6 years ago)

Author:

beidson@apple.com

Message:

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.

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/WebsiteData/WebsiteDataStore.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+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>
 

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

@@ -58 +58 @@
 static HashMap<PAL::SessionID, WebsiteDataStore*>& nonDefaultDataStores()
 {
-    RELEASE_ASSERT(isMainThread());
+    RELEASE_ASSERT(isUIThread());
     static NeverDestroyed<HashMap<PAL::SessionID, WebsiteDataStore*>> map;
     return map;