Move sourceParsed to WebScriptDebugServer
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Mar 2008 00:04:31 +0000 (00:04 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 5 Mar 2008 00:04:31 +0000 (00:04 +0000)
         Reviewed by Anders.

         * WebScriptDebugServer.cpp:
         (WebScriptDebugServer::sourceParsed): Moved here from
         WebScriptDebugger and merged with didParseSource/failedToParseSource.
         * WebScriptDebugServer.h: Removed didParseSource/failedToParseSource.
         * WebScriptDebugger.cpp: Removed sourceParsed.
         * WebScriptDebugger.h: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30772 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 f6c815b1f859001f96053717f4c07bd3c0bdaebf..748153843bcb00da726710b46121aeec2a1b5223 100644 (file)
@@ -1,3 +1,16 @@
+2008-03-04  Adam Roben  <aroben@apple.com>
+
+        Move sourceParsed to WebScriptDebugServer
+
+        Reviewed by Anders.
+
+        * WebScriptDebugServer.cpp:
+        (WebScriptDebugServer::sourceParsed): Moved here from
+        WebScriptDebugger and merged with didParseSource/failedToParseSource.
+        * WebScriptDebugServer.h: Removed didParseSource/failedToParseSource.
+        * WebScriptDebugger.cpp: Removed sourceParsed.
+        * WebScriptDebugger.h: Ditto.
+
 2008-03-04  Adam Roben  <aroben@apple.com>
 
         Move callEvent, returnEvent, and exception to WebScriptDebugServer
index 2115f88861af4732a1cb7126a230957436f3713c..fdafb6d4d3617fa417a74d6ee1fbe31d81ae75e8 100644 (file)
@@ -276,42 +276,35 @@ HRESULT STDMETHODCALLTYPE WebScriptDebugServer::didLoadMainResourceForDataSource
     return S_OK;
 }
 
-HRESULT STDMETHODCALLTYPE WebScriptDebugServer::didParseSource(
-    /* [in] */ IWebView* webView,
-    /* [in] */ BSTR sourceCode,
-    /* [in] */ UINT baseLineNumber,
-    /* [in] */ BSTR url,
-    /* [in] */ int sourceID,
-    /* [in] */ IWebFrame* webFrame)
+bool WebScriptDebugServer::sourceParsed(ExecState* exec, int sourceID, const UString& sourceURL,
+                  const UString& source, int startingLineNumber, int errorLine, const UString& /*errorMsg*/)
 {
-    if (!webView || !sourceCode || !url || !webFrame)
-        return E_FAIL;
-
-    ListenerSet listenersCopy = s_Listeners;
-    ListenerSet::iterator end = listenersCopy.end();
-    for (ListenerSet::iterator it = listenersCopy.begin(); it != end; ++it)
-        (**it).didParseSource(webView, sourceCode, baseLineNumber, url, sourceID, webFrame);
+    if (m_callingServer)
+        return true;
 
-    return S_OK;
-}
+    m_callingServer = true;
 
-HRESULT STDMETHODCALLTYPE WebScriptDebugServer::failedToParseSource(
-    /* [in] */ IWebView* webView,
-    /* [in] */ BSTR sourceCode,
-    /* [in] */ UINT baseLineNumber,
-    /* [in] */ BSTR url,
-    /* [in] */ BSTR error,
-    /* [in] */ IWebFrame* webFrame)
-{
-    if (!webView || !sourceCode || !url || !error || !webFrame)
-        return E_FAIL;
+    if (listenerCount() <= 0)
+        return true;
 
+    BString bSource = String(source);
+    BString bSourceURL = String(sourceURL);
+    
     ListenerSet listenersCopy = s_Listeners;
     ListenerSet::iterator end = listenersCopy.end();
-    for (ListenerSet::iterator it = listenersCopy.begin(); it != end; ++it)
-        (**it).failedToParseSource(webView, sourceCode, baseLineNumber, url, error, webFrame);
+    if (errorLine == -1) {
+        for (ListenerSet::iterator it = listenersCopy.begin(); it != end; ++it)
+            (**it).didParseSource(webView(exec), bSource, startingLineNumber, bSourceURL, sourceID, webFrame(exec));
+    } else {
+        // FIXME: the error var should be made with the information in the errorMsg.  It is not a simple
+        // UString to BSTR conversion there is some logic involved that I don't fully understand yet.
+        BString error(L"An Error Occurred.");
+        for (ListenerSet::iterator it = listenersCopy.begin(); it != end; ++it)
+            (**it).failedToParseSource(webView(exec), bSource, startingLineNumber, bSourceURL, error, webFrame(exec));
+    }
 
-    return S_OK;
+    m_callingServer = false;
+    return true;
 }
 
 bool WebScriptDebugServer::callEvent(ExecState* exec, int sourceID, int lineNumber, JSObject* /*function*/, const List &/*args*/)
index b0fb1cfb579464033f3570bea608e0608417dccd..16f44a25d85f3e8d7bff585f38ee8cd9863351a4 100644 (file)
@@ -82,6 +82,8 @@ public:
         /* [out, retval] */ BOOL* isPaused);
 
     // KJS::Debugger
+    bool sourceParsed(KJS::ExecState*, int sourceID, const KJS::UString& sourceURL,
+        const KJS::UString& source, int startingLineNumber, int errorLine, const KJS::UString& errorMsg);
     bool callEvent(KJS::ExecState*, int sourceID, int lineNumber, KJS::JSObject* function, const KJS::List& args);
     bool atStatement(KJS::ExecState*, int sourceID, int firstLine, int lastLine);
     bool returnEvent(KJS::ExecState*, int sourceID, int lineNumber, KJS::JSObject* function);
@@ -92,22 +94,6 @@ public:
         /* [in] */ IWebView* webView,
         /* [in] */ IWebDataSource* dataSource);
 
-    virtual HRESULT STDMETHODCALLTYPE didParseSource(
-        /* [in] */ IWebView* webView,
-        /* [in] */ BSTR sourceCode,
-        /* [in] */ UINT baseLineNumber,
-        /* [in] */ BSTR url,
-        /* [in] */ int sourceID,
-        /* [in] */ IWebFrame* webFrame);
-
-    virtual HRESULT STDMETHODCALLTYPE failedToParseSource(
-        /* [in] */ IWebView* webView,
-        /* [in] */ BSTR sourceCode,
-        /* [in] */ UINT baseLineNumber,
-        /* [in] */ BSTR url,
-        /* [in] */ BSTR error,
-        /* [in] */ IWebFrame*);
-
     virtual HRESULT STDMETHODCALLTYPE serverDidDie();
 
     void suspendProcessIfPaused();
index 8952060924b01a4892e70467137a4f3e011f681a..f283809b0096030660bb5f47e61e7238f0ddcb32 100644 (file)
@@ -72,39 +72,3 @@ WebView* webView(ExecState* exec)
     return kit(frame(exec)->page());
 }
 
-bool WebScriptDebugger::sourceParsed(ExecState* exec, int sourceId, const UString& sourceURL,
-                  const UString& source, int startingLineNumber, int errorLine, const UString& /*errorMsg*/)
-{
-    if (m_callingServer)
-        return true;
-
-    m_callingServer = true;
-
-    if (WebScriptDebugServer::listenerCount() <= 0)
-        return true;
-
-    BString bSource = String(source);
-    BString bSourceURL = String(sourceURL);
-    
-    if (errorLine == -1) {
-        WebScriptDebugServer::sharedWebScriptDebugServer()->didParseSource(webView(exec),
-            bSource,
-            startingLineNumber,
-            bSourceURL,
-            sourceId,
-            webFrame(exec));
-    } else {
-        // FIXME: the error var should be made with the information in the errorMsg.  It is not a simple
-        // UString to BSTR conversion there is some logic involved that I don't fully understand yet.
-        BString error(L"An Error Occurred.");
-        WebScriptDebugServer::sharedWebScriptDebugServer()->failedToParseSource(webView(exec),
-            bSource,
-            startingLineNumber,
-            bSourceURL,
-            error,
-            webFrame(exec));
-    }
-
-    m_callingServer = false;
-    return true;
-}
index 8c425f4ec469d1f98678a2078477bb135e898fae..97d729e1b16cea5439927ca81bc93719001d5e66 100644 (file)
@@ -49,8 +49,6 @@ WebView* webView(KJS::ExecState*);
 
 class WebScriptDebugger : public KJS::Debugger {
 public:
-    bool sourceParsed(KJS::ExecState*, int sourceId, const KJS::UString& sourceURL,
-        const KJS::UString& source, int startingLineNumber, int errorLine, const KJS::UString& errorMsg);
 
 protected:
     WebScriptDebugger();