Changeset 76392 in webkit

Timestamp:

Jan 21, 2011 2:00:52 PM (13 years ago)

Author:

pfeldman@chromium.org

Message:

2011-01-21 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: ~InspectorResourceAgent crashes on closing inspected page.
​https://bugs.webkit.org/show_bug.cgi?id=52900

• inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::inspectedPageDestroyed):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/InspectorController.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-01-21  Pavel Feldman  <pfeldman@chromium.org>
+
+        Reviewed by Yury Semikhatsky.
+
+        Web Inspector: ~InspectorResourceAgent crashes on closing inspected page.
+        https://bugs.webkit.org/show_bug.cgi?id=52900
+
+        * inspector/InspectorController.cpp:
+        (WebCore::InspectorController::~InspectorController):
+        (WebCore::InspectorController::inspectedPageDestroyed):
+
 2011-01-21  Chris Rogers  <crogers@google.com>
 

trunk/Source/WebCore/inspector/InspectorController.cpp

@@ -159 +159 @@
     ASSERT(!m_inspectedPage);
     ASSERT(!m_highlightedNode);
-
-    releaseFrontendLifetimeAgents();
-
-    m_injectedScriptHost->disconnectController();
 }
 
@@ -179 +175 @@
     ASSERT(m_inspectedPage);
     m_inspectedPage = 0;
+
+    releaseFrontendLifetimeAgents();
+    m_injectedScriptHost->disconnectController();
 
     m_client->inspectorDestroyed();