Web Inspector: JSGlobalRuntimeAgent unintentionally overrides InspectorRuntimeAgent destruction handling

https://bugs.webkit.org/show_bug.cgi?id=149537

Patch by Joseph Pecoraro <[email protected]> on 2015-09-28
Reviewed by Darin Adler.

* inspector/agents/JSGlobalObjectRuntimeAgent.h:
* inspector/agents/JSGlobalObjectRuntimeAgent.cpp:
(Inspector::JSGlobalObjectRuntimeAgent::willDestroyFrontendAndBackend): Deleted.
Do not override method, super class implementation is sufficient.

* JavaScriptCore.xcodeproj/project.pbxproj:
Fix file ordering.

* inspector/agents/InspectorDebuggerAgent.h:
Remove unused member variable.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@190301 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Author: [email protected]

Source/JavaScriptCore/ChangeLog

@@ -1,3 +1,21 @@
+2015-09-28  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: JSGlobalRuntimeAgent unintentionally overrides InspectorRuntimeAgent destruction handling
+        https://bugs.webkit.org/show_bug.cgi?id=149537
+
+        Reviewed by Darin Adler.
+
+        * inspector/agents/JSGlobalObjectRuntimeAgent.h:
+        * inspector/agents/JSGlobalObjectRuntimeAgent.cpp:
+        (Inspector::JSGlobalObjectRuntimeAgent::willDestroyFrontendAndBackend): Deleted.
+        Do not override method, super class implementation is sufficient.
+
+        * JavaScriptCore.xcodeproj/project.pbxproj:
+        Fix file ordering.
+
+        * inspector/agents/InspectorDebuggerAgent.h:
+        Remove unused member variable.
+
 2015-09-28  [email protected]  <[email protected]@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
 
         [ES6] Implement tail calls in the FTL

Source/JavaScriptCore/inspector/agents/InspectorDebuggerAgent.h

@@ -173,7 +173,6 @@ class JS_EXPORT_PRIVATE InspectorDebuggerAgent : public InspectorAgentBase, publ
     bool m_javaScriptPauseScheduled { false };
     bool m_hasExceptionValue { false };
     bool m_didPauseStopwatch { false };
-    RefPtr<WTF::Stopwatch> m_stopwatch;
 };
 
 } // namespace Inspector

Source/JavaScriptCore/inspector/agents/JSGlobalObjectRuntimeAgent.cpp

@@ -46,10 +46,6 @@ void JSGlobalObjectRuntimeAgent::didCreateFrontendAndBackend(FrontendRouter*, Ba
 {
 }
 
-void JSGlobalObjectRuntimeAgent::willDestroyFrontendAndBackend(DisconnectReason)
-{
-}
-
 VM& JSGlobalObjectRuntimeAgent::globalVM()
 {
     return m_globalObject.vm();

Source/JavaScriptCore/inspector/agents/JSGlobalObjectRuntimeAgent.h

@@ -39,7 +39,6 @@ class JSGlobalObjectRuntimeAgent final : public InspectorRuntimeAgent {
     JSGlobalObjectRuntimeAgent(JSAgentContext&);
 
     virtual void didCreateFrontendAndBackend(FrontendRouter*, BackendDispatcher*) override;
-    virtual void willDestroyFrontendAndBackend(DisconnectReason) override;
 
     virtual JSC::VM& globalVM() override;
     virtual InjectedScript injectedScriptForEval(ErrorString&, const int* executionContextId) override;