Rename WebCoreScriptDebuggerImp to WebScriptDebugger
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Mar 2008 16:15:08 +0000 (16:15 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Mar 2008 16:15:08 +0000 (16:15 +0000)
        Reviewed by Kevin M.

        * WebView/WebCoreScriptDebuggerImp.h:
        * WebView/WebCoreScriptDebuggerImp.mm:
        * WebView/WebFrame.mm:
        (-[WebFrame _attachScriptDebugger]):
        * WebView/WebFrameInternal.h:

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

WebKit/mac/ChangeLog
WebKit/mac/WebView/WebCoreScriptDebuggerImp.h
WebKit/mac/WebView/WebCoreScriptDebuggerImp.mm
WebKit/mac/WebView/WebFrame.mm
WebKit/mac/WebView/WebFrameInternal.h

index 4f0290a6b02a7c8abfbf5992959dd85b1fb6bf61..83ce67d1c8bbdadb04564fd9b813e3a5f019dc31 100644 (file)
@@ -1,3 +1,15 @@
+2008-03-05  Adam Roben  <aroben@apple.com>
+
+        Rename WebCoreScriptDebuggerImp to WebScriptDebugger
+
+        Reviewed by Kevin M.
+
+        * WebView/WebCoreScriptDebuggerImp.h:
+        * WebView/WebCoreScriptDebuggerImp.mm:
+        * WebView/WebFrame.mm:
+        (-[WebFrame _attachScriptDebugger]):
+        * WebView/WebFrameInternal.h:
+
 2008-03-05  Adam Roben  <aroben@apple.com>
 
         Remove WebScriptDebugger
index 8975758ba62c60a81088195805d68ec3532916ff..76dfed00ba8106bc32f978530512fc7b3be7a860 100644 (file)
@@ -26,8 +26,6 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-// FIXME: This file and class should be renamed WebScriptDebugger
-
 #ifndef WebCoreScriptDebuggerImp_h
 #define WebCoreScriptDebuggerImp_h
 
@@ -48,9 +46,9 @@ namespace KJS {
 
 NSString *toNSString(const KJS::UString&);
 
-class WebCoreScriptDebuggerImp : public KJS::Debugger {
+class WebScriptDebugger : public KJS::Debugger {
 public:
-    WebCoreScriptDebuggerImp(KJS::JSGlobalObject*);
+    WebScriptDebugger(KJS::JSGlobalObject*);
 
     virtual bool sourceParsed(KJS::ExecState*, int sourceID, const KJS::UString& sourceURL, const KJS::UString& source, int lineNumber, int errorLine, const KJS::UString& errorMsg);
     virtual bool callEvent(KJS::ExecState*, int sourceID, int lineNumber, KJS::JSObject* function, const KJS::List& args);
index 4cce739b73b9149b8291938eb8f3408a3af9736f..fedb61de6fcd60842d2f0cebce7b58107d24334f 100644 (file)
@@ -26,8 +26,6 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-// FIXME: This file and class should be renamed WebScriptDebugger
-
 #include "WebCoreScriptDebuggerImp.h"
 
 #include "WebFrameInternal.h"
@@ -70,7 +68,7 @@ static WebFrame *toWebFrame(ExecState* state)
     return kit(window->impl()->frame());
 }
 
-WebCoreScriptDebuggerImp::WebCoreScriptDebuggerImp(JSGlobalObject* globalObject)
+WebScriptDebugger::WebScriptDebugger(JSGlobalObject* globalObject)
     : m_callingDelegate(false)
 {
     attach(globalObject);
@@ -79,7 +77,7 @@ WebCoreScriptDebuggerImp::WebCoreScriptDebuggerImp(JSGlobalObject* globalObject)
 }
 
 // callbacks - relay to delegate
-bool WebCoreScriptDebuggerImp::sourceParsed(ExecState* state, int sourceID, const UString& url, const UString& source, int lineNumber, int errorLine, const UString& errorMsg)
+bool WebScriptDebugger::sourceParsed(ExecState* state, int sourceID, const UString& url, const UString& source, int lineNumber, int errorLine, const UString& errorMsg)
 {
     if (m_callingDelegate)
         return true;
@@ -112,7 +110,7 @@ bool WebCoreScriptDebuggerImp::sourceParsed(ExecState* state, int sourceID, cons
     return true;
 }
 
-bool WebCoreScriptDebuggerImp::callEvent(ExecState* state, int sourceID, int lineNumber, JSObject*, const List&)
+bool WebScriptDebugger::callEvent(ExecState* state, int sourceID, int lineNumber, JSObject*, const List&)
 {
     if (m_callingDelegate)
         return true;
@@ -133,7 +131,7 @@ bool WebCoreScriptDebuggerImp::callEvent(ExecState* state, int sourceID, int lin
     return true;
 }
 
-bool WebCoreScriptDebuggerImp::atStatement(ExecState* state, int sourceID, int lineNumber, int)
+bool WebScriptDebugger::atStatement(ExecState* state, int sourceID, int lineNumber, int)
 {
     if (m_callingDelegate)
         return true;
@@ -151,7 +149,7 @@ bool WebCoreScriptDebuggerImp::atStatement(ExecState* state, int sourceID, int l
     return true;
 }
 
-bool WebCoreScriptDebuggerImp::returnEvent(ExecState* state, int sourceID, int lineNumber, JSObject*)
+bool WebScriptDebugger::returnEvent(ExecState* state, int sourceID, int lineNumber, JSObject*)
 {
     if (m_callingDelegate)
         return true;
@@ -171,7 +169,7 @@ bool WebCoreScriptDebuggerImp::returnEvent(ExecState* state, int sourceID, int l
     return true;
 }
 
-bool WebCoreScriptDebuggerImp::exception(ExecState* state, int sourceID, int lineNumber, JSValue*)
+bool WebScriptDebugger::exception(ExecState* state, int sourceID, int lineNumber, JSValue*)
 {
     if (m_callingDelegate)
         return true;
index 43c5505b63a907db76d9d63eebf6725930a4747d..c5e2ca38684491642c44481f6316e13b7d70bf69 100644 (file)
@@ -380,7 +380,7 @@ WebView *getWebView(WebFrame *webFrame)
     if (!globalObject)
         return;
 
-    _private->scriptDebugger = new WebCoreScriptDebuggerImp(globalObject);
+    _private->scriptDebugger = new WebScriptDebugger(globalObject);
 }
 
 - (void)_detachScriptDebugger
index ea913c830187c46d6dcbbf1c54692da6fc2697e1..d6828256d8e8811b7e33877adc5dc06b8b451c3c 100644 (file)
@@ -43,7 +43,7 @@
 @class WebFrameView;
 @class WebFrameBridge;
 @class WebHistoryItem;
-class WebCoreScriptDebuggerImp;
+class WebScriptDebugger;
 
 #ifdef __cplusplus
 
@@ -96,7 +96,7 @@ WebView *getWebView(WebFrame *webFrame);
 @public
     WebFrameView *webFrameView;
     WebFrameBridge *bridge;
-    WebCoreScriptDebuggerImp* scriptDebugger;
+    WebScriptDebugger* scriptDebugger;
     id internalLoadDelegate;
 }
 @end