Title: [86955] trunk/Source/WebCore
Revision
86955
Author
[email protected]
Date
2011-05-20 08:18:40 -0700 (Fri, 20 May 2011)

Log Message

2011-05-20  Yury Semikhatsky  <[email protected]>

        Reviewed by Pavel Feldman.

        Web Inspector: use RefPtr instead of OwnPtr to store InspectorBackendDispatcher
        https://bugs.webkit.org/show_bug.cgi?id=61188

        * inspector/WorkerInspectorController.cpp:
        (WebCore::WorkerInspectorController::connectFrontend):
        (WebCore::WorkerInspectorController::disconnectFrontend):
        * inspector/WorkerInspectorController.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (86954 => 86955)


--- trunk/Source/WebCore/ChangeLog	2011-05-20 15:04:32 UTC (rev 86954)
+++ trunk/Source/WebCore/ChangeLog	2011-05-20 15:18:40 UTC (rev 86955)
@@ -1,3 +1,15 @@
+2011-05-20  Yury Semikhatsky  <[email protected]>
+
+        Reviewed by Pavel Feldman.
+
+        Web Inspector: use RefPtr instead of OwnPtr to store InspectorBackendDispatcher
+        https://bugs.webkit.org/show_bug.cgi?id=61188
+
+        * inspector/WorkerInspectorController.cpp:
+        (WebCore::WorkerInspectorController::connectFrontend):
+        (WebCore::WorkerInspectorController::disconnectFrontend):
+        * inspector/WorkerInspectorController.h:
+
 2011-05-20  Tonis Tiigi  <[email protected]>
 
         Reviewed by Pavel Feldman.

Modified: trunk/Source/WebCore/inspector/WorkerInspectorController.cpp (86954 => 86955)


--- trunk/Source/WebCore/inspector/WorkerInspectorController.cpp	2011-05-20 15:04:32 UTC (rev 86954)
+++ trunk/Source/WebCore/inspector/WorkerInspectorController.cpp	2011-05-20 15:18:40 UTC (rev 86955)
@@ -98,7 +98,7 @@
     ASSERT(!m_frontend);
     m_state->unmute();
     m_frontend = adoptPtr(new InspectorFrontend(channel));
-    m_backendDispatcher = adoptPtr(new InspectorBackendDispatcher(
+    m_backendDispatcher = adoptRef(new InspectorBackendDispatcher(
         channel,
 #if ENABLE(OFFLINE_WEB_APPLICATIONS)
         0, // InspectorApplicationCacheAgent
@@ -138,6 +138,7 @@
 {
     if (!m_frontend)
         return;
+    m_backendDispatcher->clearFrontend();
     m_backendDispatcher.clear();
     // Destroying agents would change the state, but we don't want that.
     // Pre-disconnect state will be used to restore inspector agents.

Modified: trunk/Source/WebCore/inspector/WorkerInspectorController.h (86954 => 86955)


--- trunk/Source/WebCore/inspector/WorkerInspectorController.h	2011-05-20 15:04:32 UTC (rev 86954)
+++ trunk/Source/WebCore/inspector/WorkerInspectorController.h	2011-05-20 15:18:40 UTC (rev 86955)
@@ -37,6 +37,7 @@
 #include <wtf/Forward.h>
 #include <wtf/Noncopyable.h>
 #include <wtf/OwnPtr.h>
+#include <wtf/RefPtr.h>
 
 namespace WebCore {
 
@@ -71,7 +72,7 @@
     OwnPtr<InspectorRuntimeAgent> m_runtimeAgent;
 
     OwnPtr<InspectorFrontend> m_frontend;
-    OwnPtr<InspectorBackendDispatcher> m_backendDispatcher;
+    RefPtr<InspectorBackendDispatcher> m_backendDispatcher;
 };
 
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to