Changeset 233582 in webkit

Timestamp:

Jul 6, 2018, 11:25:59 AM (8 years ago)

Author:

Chris Dumez

Message:

[iOS] Assertion failure in WebProcessProxy::allProcesses() (isMainThread())
​https://bugs.webkit.org/show_bug.cgi?id=187394

Reviewed by Dan Bernstein.

Use RunLoop::isMain() instead of isMainThread() in the assertion to fix issues for
applications using both WebKit and WebKitLegacy.

• UIProcess/WebProcessProxy.cpp:

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/WebProcessProxy.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2018-07-06  Chris Dumez  <cdumez@apple.com>
+
+        [iOS] Assertion failure in WebProcessProxy::allProcesses() (isMainThread())
+        https://bugs.webkit.org/show_bug.cgi?id=187394
+
+        Reviewed by Dan Bernstein.
+
+        Use RunLoop::isMain() instead of isMainThread() in the assertion to fix issues for
+        applications using both WebKit and WebKitLegacy.
+
+        * UIProcess/WebProcessProxy.cpp:
+
 2018-07-06  Daniel Bates  <dabates@apple.com>
 

trunk/Source/WebKit/UIProcess/WebProcessProxy.cpp

@@ -83 +83 @@
 static HashMap<ProcessIdentifier, WebProcessProxy*>& allProcesses()
 {
-    ASSERT(isMainThread());
+    ASSERT(RunLoop::isMain());
     static NeverDestroyed<HashMap<ProcessIdentifier, WebProcessProxy*>> map;
     return map;