Remove -enteredFrame:, -leavingFrame:, and -exceptionRaised: from WebScriptDe...
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Mar 2008 16:13:23 +0000 (16:13 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Mar 2008 16:13:23 +0000 (16:13 +0000)
        Reviewed by Kevin M.

        * WebView/WebCoreScriptDebuggerImp.mm:
        (WebCoreScriptDebuggerImp::WebCoreScriptDebuggerImp): Changed to call
        trhough to callEvent instead of duplicating its code here.
        (WebCoreScriptDebuggerImp::callEvent): Moved code from
        -[WebScriptDebugger enteredFrame:sourceId:line:] here.
        (WebCoreScriptDebuggerImp::returnEvent): Moved code from
        -[WebScriptDebugger leavingFrame:sourceId:line:] here.
        (WebCoreScriptDebuggerImp::exception): Moved code from
        -[WebScriptDebugger exceptionRaised:sourceId:line:] here.
        * WebView/WebScriptDebugDelegate.mm: Removed -enteredFrame:,
        -leavingFrame:, and -exceptionRaised:.
        * WebView/WebScriptDebugDelegatePrivate.h: Ditto.

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

WebKit/mac/ChangeLog
WebKit/mac/WebView/WebCoreScriptDebuggerImp.mm
WebKit/mac/WebView/WebScriptDebugDelegate.mm
WebKit/mac/WebView/WebScriptDebugDelegatePrivate.h

index 5954c4ccfc5d7cda4075c71757aa66efd94f7749..80015a33c817770b245af674404f120fa9b6a711 100644 (file)
@@ -1,3 +1,22 @@
+2008-03-05  Adam Roben  <aroben@apple.com>
+
+        Remove -enteredFrame:, -leavingFrame:, and -exceptionRaised: from WebScriptDebugger
+
+        Reviewed by Kevin M.
+
+        * WebView/WebCoreScriptDebuggerImp.mm:
+        (WebCoreScriptDebuggerImp::WebCoreScriptDebuggerImp): Changed to call
+        trhough to callEvent instead of duplicating its code here.
+        (WebCoreScriptDebuggerImp::callEvent): Moved code from
+        -[WebScriptDebugger enteredFrame:sourceId:line:] here.
+        (WebCoreScriptDebuggerImp::returnEvent): Moved code from
+        -[WebScriptDebugger leavingFrame:sourceId:line:] here.
+        (WebCoreScriptDebuggerImp::exception): Moved code from
+        -[WebScriptDebugger exceptionRaised:sourceId:line:] here.
+        * WebView/WebScriptDebugDelegate.mm: Removed -enteredFrame:,
+        -leavingFrame:, and -exceptionRaised:.
+        * WebView/WebScriptDebugDelegatePrivate.h: Ditto.
+
 2008-03-05  Adam Roben  <aroben@apple.com>
 
         Remove -[WebScriptDebugger hitStatement:sourceId:line:]
index d8994d228496582555262a38d1e238348e41b1e7..2e24369821751c26538c403f083e6f392b35138b 100644 (file)
@@ -58,12 +58,12 @@ static NSURL *toNSURL(const UString& s)
 
 WebCoreScriptDebuggerImp::WebCoreScriptDebuggerImp(WebScriptDebugger *debugger, JSGlobalObject* globalObject)
     : m_debugger(debugger)
+    , m_callingDelegate(false)
+    , m_topCallFrame(nil)
 {
-    m_callingDelegate = true;
-    m_topCallFrame = [m_debugger enterFrame:globalObject->globalExec()];
     attach(globalObject);
-    [m_debugger enteredFrame:m_topCallFrame sourceId:-1 line:-1];
-    m_callingDelegate = false;
+    List emptyList;
+    callEvent(globalObject->globalExec(), -1, -1, 0, emptyList);
 }
 
 // callbacks - relay to delegate
@@ -86,7 +86,13 @@ bool WebCoreScriptDebuggerImp::callEvent(ExecState* state, int sourceID, int lin
 
     m_callingDelegate = true;
     m_topCallFrame = [m_debugger enterFrame:state];
-    [m_debugger enteredFrame:m_topCallFrame sourceId:sourceID line:lineNumber];
+
+    WebFrame *webFrame = [m_debugger webFrame];
+    WebView *webView = [webFrame webView];
+    [[webView _scriptDebugDelegateForwarder] webView:webView didEnterCallFrame:m_topCallFrame sourceId:sourceID line:lineNumber forWebFrame:webFrame];
+    if ([WebScriptDebugServer listenerCount])
+        [[WebScriptDebugServer sharedScriptDebugServer] webView:webView didEnterCallFrame:m_topCallFrame sourceId:sourceID line:lineNumber forWebFrame:webFrame];
+
     m_callingDelegate = false;
 
     return true;
@@ -116,7 +122,13 @@ bool WebCoreScriptDebuggerImp::returnEvent(ExecState* state, int sourceID, int l
         return true;
 
     m_callingDelegate = true;
-    [m_debugger leavingFrame:m_topCallFrame sourceId:sourceID line:lineNumber];
+
+    WebFrame *webFrame = [m_debugger webFrame];
+    WebView *webView = [webFrame webView];
+    [[webView _scriptDebugDelegateForwarder] webView:webView willLeaveCallFrame:m_topCallFrame sourceId:sourceID line:lineNumber forWebFrame:webFrame];
+    if ([WebScriptDebugServer listenerCount])
+        [[WebScriptDebugServer sharedScriptDebugServer] webView:webView willLeaveCallFrame:m_topCallFrame sourceId:sourceID line:lineNumber forWebFrame:webFrame];
+
     m_topCallFrame = [m_debugger leaveFrame];
     m_callingDelegate = false;
 
@@ -129,7 +141,13 @@ bool WebCoreScriptDebuggerImp::exception(ExecState* state, int sourceID, int lin
         return true;
 
     m_callingDelegate = true;
-    [m_debugger exceptionRaised:m_topCallFrame sourceId:sourceID line:lineNumber];
+
+    WebFrame *webFrame = [m_debugger webFrame];
+    WebView *webView = [webFrame webView];
+    [[webView _scriptDebugDelegateForwarder] webView:webView exceptionWasRaised:m_topCallFrame sourceId:sourceID line:lineNumber forWebFrame:webFrame];
+    if ([WebScriptDebugServer listenerCount])
+        [[WebScriptDebugServer sharedScriptDebugServer] webView:webView exceptionWasRaised:m_topCallFrame sourceId:sourceID line:lineNumber forWebFrame:webFrame];
+
     m_callingDelegate = false;
 
     return true;
index 82ce80d7e6d41c435c93e57c0d1ff3d986497e3b..23d8caca7bc577602d467f25f51b31fd7da5e4a3 100644 (file)
@@ -118,30 +118,6 @@ NSString * const WebScriptErrorLineNumberKey = @"WebScriptErrorLineNumber";
     }
 }
 
-- (void)enteredFrame:(WebScriptCallFrame *)frame sourceId:(int)sid line:(int)lineno
-{
-    WebView *webView = [_webFrame webView];
-    [[webView _scriptDebugDelegateForwarder] webView:webView didEnterCallFrame:frame sourceId:sid line:lineno forWebFrame:_webFrame];
-    if ([WebScriptDebugServer listenerCount])
-        [[WebScriptDebugServer sharedScriptDebugServer] webView:webView didEnterCallFrame:frame sourceId:sid line:lineno forWebFrame:_webFrame];
-}
-
-- (void)leavingFrame:(WebScriptCallFrame *)frame sourceId:(int)sid line:(int)lineno
-{
-    WebView *webView = [_webFrame webView];
-    [[webView _scriptDebugDelegateForwarder] webView:webView willLeaveCallFrame:frame sourceId:sid line:lineno forWebFrame:_webFrame];
-    if ([WebScriptDebugServer listenerCount])
-        [[WebScriptDebugServer sharedScriptDebugServer] webView:webView willLeaveCallFrame:frame sourceId:sid line:lineno forWebFrame:_webFrame];
-}
-
-- (void)exceptionRaised:(WebScriptCallFrame *)frame sourceId:(int)sid line:(int)lineno
-{
-    WebView *webView = [_webFrame webView];
-    [[webView _scriptDebugDelegateForwarder] webView:webView exceptionWasRaised:frame sourceId:sid line:lineno forWebFrame:_webFrame];
-    if ([WebScriptDebugServer listenerCount])
-        [[WebScriptDebugServer sharedScriptDebugServer] webView:webView exceptionWasRaised:frame sourceId:sid line:lineno forWebFrame:_webFrame];
-}
-
 @end
 
 @interface WebScriptCallFramePrivate : NSObject {
index 12afc043806db25d63a8fd865d0ef974648068a8..ff0436bd4448b62e6f247379421bd585c70334d3 100644 (file)
@@ -50,8 +50,5 @@ class WebCoreScriptDebuggerImp;
 
 // debugger callbacks
 - (void)parsedSource:(NSString *)source fromURL:(NSURL *)url sourceId:(int)sid startLine:(int)startLine errorLine:(int)errorLine errorMessage:(NSString *)errorMessage;
-- (void)enteredFrame:(WebScriptCallFrame *)frame sourceId:(int)sid line:(int)lineno;
-- (void)leavingFrame:(WebScriptCallFrame *)frame sourceId:(int)sid line:(int)lineno;
-- (void)exceptionRaised:(WebScriptCallFrame *)frame sourceId:(int)sid line:(int)lineno;
 
 @end