Changeset 71129 in webkit

Timestamp:

Nov 2, 2010 10:51:16 AM (13 years ago)

Author:

andersca@apple.com

Message:

Assertion hit in WebProcessProxy::~WebProcessProxy()
​https://bugs.webkit.org/show_bug.cgi?id=48849
<rdar://problem/8617676>

Reviewed by John Sullivan.

It is perfectly valid for the page map to contain pages if the web process proxy
goes away, this can happen if the process crashes for example.

• UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::~WebProcessProxy):
Remove the assertion.

Location:

trunk/WebKit2

Files:

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

trunk/WebKit2/ChangeLog

@@ +1 @@
+2010-11-02  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by John Sullivan.
+
+        Assertion hit in WebProcessProxy::~WebProcessProxy()
+        https://bugs.webkit.org/show_bug.cgi?id=48849
+        <rdar://problem/8617676>
+
+        It is perfectly valid for the page map to contain pages if the web process proxy
+        goes away, this can happen if the process crashes for example.
+
+        * UIProcess/WebProcessProxy.cpp:
+        (WebKit::WebProcessProxy::~WebProcessProxy):
+        Remove the assertion.
+
 2010-11-02  Christian Sejersen  <christian.webkit@gmail.com>
 

trunk/WebKit2/UIProcess/WebProcessProxy.cpp

@@ -64 +64 @@
 WebProcessProxy::~WebProcessProxy()
 {
-    ASSERT(m_pageMap.isEmpty());
-
     if (m_connection)
         m_connection->invalidate();