Devtools: Move runMessageLoopOnPause and other methods on ScriptDebugServer::Client

Source/bindings/core/v8/WorkerScriptDebugServer.h

 /*
  * Copyright (c) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef WorkerScriptDebugServer_h
 #define WorkerScriptDebugServer_h
 
-#include "bindings/core/v8/ScriptDebugServer.h"
+#include "core/inspector/PerIsolateDebuggerClient.h"
 #include <v8.h>
 
 namespace blink {
 
 class WorkerGlobalScope;
 
-class WorkerScriptDebugServer final : public ScriptDebugServer {
+class WorkerScriptDebugServer final : public PerIsolateDebuggerClient {
     WTF_MAKE_NONCOPYABLE(WorkerScriptDebugServer);
 public:
     static PassOwnPtrWillBeRawPtr<WorkerScriptDebugServer> create(WorkerGlobalScope* workerGlobalScope)
     {
         return adoptPtrWillBeNoop(new WorkerScriptDebugServer(workerGlobalScope));
     }
 
     ~WorkerScriptDebugServer() override { }
-    DECLARE_VIRTUAL_TRACE();
 
     static void setContextDebugData(v8::Local<v8::Context>);
     void addListener(ScriptDebugListener*);
     void removeListener(ScriptDebugListener*);
+    ScriptDebugListener* getDebugListenerForContext(v8::Local<v8::Context>);
+    void runMessageLoopOnPause(v8::Local<v8::Context>);
+    void quitMessageLoopOnPause();
 
 private:
     explicit WorkerScriptDebugServer(WorkerGlobalScope*);
 
-    ScriptDebugListener* getDebugListenerForContext(v8::Local<v8::Context>) override;
-    void runMessageLoopOnPause(v8::Local<v8::Context>) override;

[yurys, 2015/05/08 10:59:15]

These methods should also stay private for clarity.

[sergeyv, 2015/05/08 13:59:14]

Done.

-    void quitMessageLoopOnPause() override;
-
     ScriptDebugListener* m_listener;
     RawPtrWillBeMember<WorkerGlobalScope> m_workerGlobalScope;
 };
 
 } // namespace blink
 
 #endif // WorkerScriptDebugServer_h