Start to merge WebScriptDebugger and WebScriptDebugServer
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Mar 2008 00:02:54 +0000 (00:02 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Mar 2008 00:02:54 +0000 (00:02 +0000)
         WebScriptDebugServer now inherits from WebScriptDebugger.

         Reviewed by Anders.

         * WebScriptDebugServer.cpp:
         (WebScriptDebugServer::pageCreated): Use sharedWebScriptDebugServer
         instead of WebScriptDebugger::shared.
         (WebScriptDebugServer::addListener): Ditto.
         (WebScriptDebugServer::removeListener): Ditto.
         * WebScriptDebugServer.h: Changed to inherit from WebScriptDebugger.
         * WebScriptDebugger.cpp: Removed shared method.
         * WebScriptDebugger.h: Made constructor/destructor protected, and
         removed shared method.

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

WebKit/win/ChangeLog
WebKit/win/WebScriptDebugServer.cpp
WebKit/win/WebScriptDebugServer.h
WebKit/win/WebScriptDebugger.cpp
WebKit/win/WebScriptDebugger.h

index b8b5f169374b2ed9626c4581254b0433c5dd987f..c4e2ff89cf4c90985fa9cabe9c6e8381e5dd9b55 100644 (file)
@@ -1,3 +1,21 @@
+2008-03-04  Adam Roben  <aroben@apple.com>
+
+        Start to merge WebScriptDebugger and WebScriptDebugServer
+
+        WebScriptDebugServer now inherits from WebScriptDebugger.
+
+        Reviewed by Anders.
+
+        * WebScriptDebugServer.cpp:
+        (WebScriptDebugServer::pageCreated): Use sharedWebScriptDebugServer
+        instead of WebScriptDebugger::shared.
+        (WebScriptDebugServer::addListener): Ditto.
+        (WebScriptDebugServer::removeListener): Ditto.
+        * WebScriptDebugServer.h: Changed to inherit from WebScriptDebugger.
+        * WebScriptDebugger.cpp: Removed shared method.
+        * WebScriptDebugger.h: Made constructor/destructor protected, and
+        removed shared method.
+
 2008-03-04  Adam Roben  <aroben@apple.com>
 
         Use WebCore's new debugger-attaching code
index 30c2cd72cd7ffac3f8bac182b3fe80128196b7da..8794e1e1670dd853c5d7d99383504d6c6ae7fb49 100644 (file)
@@ -83,7 +83,7 @@ void WebScriptDebugServer::pageCreated(Page* page)
     ASSERT_ARG(page, page);
 
     if (s_ListenerCount > 0)
-        page->setDebugger(&WebScriptDebugger::shared());
+        page->setDebugger(sharedWebScriptDebugServer());
 }
 
 // IUnknown -------------------------------------------------------------------
@@ -140,7 +140,7 @@ HRESULT STDMETHODCALLTYPE WebScriptDebugServer::addListener(
         return E_POINTER;
 
     if (!s_ListenerCount)
-        Page::setDebuggerForAllPages(&WebScriptDebugger::shared());
+        Page::setDebuggerForAllPages(sharedWebScriptDebugServer());
 
     ++s_ListenerCount;
     s_Listeners.add(listener);
index d866b4b3ed2f9a1fff1bf842be2a0121d2a9fd78..368091fef3e2ed5e16870ba7a031846d88ef53c5 100644 (file)
@@ -27,6 +27,7 @@
 #define WebScriptDebugServer_H
 
 #include "WebKit.h"
+#include "WebScriptDebugger.h"
 
 #include <wtf/HashSet.h>
 #pragma warning(push, 0)
@@ -39,8 +40,7 @@ namespace WebCore {
 
 interface IWebView;
 
-class WebScriptDebugServer : public IWebScriptDebugServer, public IWebScriptDebugListener
-{
+class WebScriptDebugServer : public IWebScriptDebugServer, public IWebScriptDebugListener, public WebScriptDebugger {
 public:
     static WebScriptDebugServer* createInstance();
     static WebScriptDebugServer* sharedWebScriptDebugServer();
index 0e57767aab17cdb0f989f52834a0067a3dce7773..287fc05576decb073dcc5186ae31aba692812fcb 100644 (file)
 using namespace WebCore;
 using namespace KJS;
 
-WebScriptDebugger& WebScriptDebugger::shared()
-{
-    static WebScriptDebugger debugger;
-    return debugger;
-}
-
 WebScriptDebugger::WebScriptDebugger()
     : m_callingServer(false)
 {
index f5ddf94477315bd2c5497ced4ae03571d556a9b9..5d5a610b2399235685ff14a459ae3d5921cba6f3 100644 (file)
@@ -46,8 +46,6 @@ namespace KJS {
 
 class WebScriptDebugger : public KJS::Debugger {
 public:
-    static WebScriptDebugger& shared();
-
     bool sourceParsed(KJS::ExecState*, int sourceId, const KJS::UString& sourceURL,
         const KJS::UString& source, int startingLineNumber, int errorLine, const KJS::UString& errorMsg);
 
@@ -56,10 +54,11 @@ public:
     bool returnEvent(KJS::ExecState*, int sourceId, int lineno, KJS::JSObject* function);
     bool exception(KJS::ExecState*, int sourceId, int lineno, KJS::JSValue* exception);
 
-private:
+protected:
     WebScriptDebugger();
     ~WebScriptDebugger();
 
+private:
     bool m_callingServer;
     WebFrame* m_frame;
     COMPtr<IWebView> m_webView;