[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

podivilov at chromium.org podivilov at chromium.org
Wed Dec 22 15:10:01 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit f1d995416d4c4c326f6b70b63260e6e4ba26b279
Author: podivilov at chromium.org <podivilov at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Oct 28 20:13:56 2010 +0000

    2010-10-28  Pavel Podivilov  <podivilov at chromium.org>
    
            Reviewed by Pavel Feldman.
    
            Web Inspector: fix pausing on next JavaScript statement
            https://bugs.webkit.org/show_bug.cgi?id=48534
    
            * inspector/InspectorDebuggerAgent.cpp:
            (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent):
            (WebCore::InspectorDebuggerAgent::schedulePauseOnNextStatement):
            (WebCore::InspectorDebuggerAgent::cancelPauseOnNextStatement):
            (WebCore::InspectorDebuggerAgent::pause):
            (WebCore::InspectorDebuggerAgent::didPause):
            * inspector/InspectorDebuggerAgent.h:
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@70799 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index a1b75c6..dcc5f92 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2010-10-28  Pavel Podivilov  <podivilov at chromium.org>
+
+        Reviewed by Pavel Feldman.
+
+        Web Inspector: fix pausing on next JavaScript statement
+        https://bugs.webkit.org/show_bug.cgi?id=48534
+
+        * inspector/InspectorDebuggerAgent.cpp:
+        (WebCore::InspectorDebuggerAgent::InspectorDebuggerAgent):
+        (WebCore::InspectorDebuggerAgent::schedulePauseOnNextStatement):
+        (WebCore::InspectorDebuggerAgent::cancelPauseOnNextStatement):
+        (WebCore::InspectorDebuggerAgent::pause):
+        (WebCore::InspectorDebuggerAgent::didPause):
+        * inspector/InspectorDebuggerAgent.h:
+
 2010-10-28  Yuzo Fujishima  <yuzo at google.com>
 
         Reviewed by David Hyatt.
diff --git a/WebCore/inspector/InspectorDebuggerAgent.cpp b/WebCore/inspector/InspectorDebuggerAgent.cpp
index a7eacf6..ebd03e6 100644
--- a/WebCore/inspector/InspectorDebuggerAgent.cpp
+++ b/WebCore/inspector/InspectorDebuggerAgent.cpp
@@ -62,6 +62,7 @@ InspectorDebuggerAgent::InspectorDebuggerAgent(InspectorController* inspectorCon
     , m_frontend(frontend)
     , m_pausedScriptState(0)
     , m_breakpointsLoaded(false)
+    , m_javaScriptPauseScheduled(false)
 {
 }
 
@@ -144,6 +145,8 @@ void InspectorDebuggerAgent::getScriptSource(const String& sourceID, String* scr
 
 void InspectorDebuggerAgent::schedulePauseOnNextStatement(DebuggerEventType type, PassRefPtr<InspectorValue> data)
 {
+    if (m_javaScriptPauseScheduled)
+        return;
     m_breakProgramDetails = InspectorObject::create();
     m_breakProgramDetails->setNumber("eventType", type);
     m_breakProgramDetails->setValue("eventData", data);
@@ -152,6 +155,8 @@ void InspectorDebuggerAgent::schedulePauseOnNextStatement(DebuggerEventType type
 
 void InspectorDebuggerAgent::cancelPauseOnNextStatement()
 {
+    if (m_javaScriptPauseScheduled)
+        return;
     m_breakProgramDetails = 0;
     ScriptDebugServer::shared().setPauseOnNextStatement(false);
 }
@@ -159,6 +164,7 @@ void InspectorDebuggerAgent::cancelPauseOnNextStatement()
 void InspectorDebuggerAgent::pause()
 {
     schedulePauseOnNextStatement(JavaScriptPauseEventType, InspectorObject::create());
+    m_javaScriptPauseScheduled = true;
 }
 
 void InspectorDebuggerAgent::resume()
@@ -309,6 +315,7 @@ void InspectorDebuggerAgent::didPause(ScriptState* scriptState)
     m_breakProgramDetails->setValue("callFrames", currentCallFrames());
 
     m_frontend->pausedScript(m_breakProgramDetails);
+    m_javaScriptPauseScheduled = false;
 }
 
 void InspectorDebuggerAgent::didContinue()
diff --git a/WebCore/inspector/InspectorDebuggerAgent.h b/WebCore/inspector/InspectorDebuggerAgent.h
index 3825042..9501ac8 100644
--- a/WebCore/inspector/InspectorDebuggerAgent.h
+++ b/WebCore/inspector/InspectorDebuggerAgent.h
@@ -106,6 +106,7 @@ private:
     bool m_breakpointsLoaded;
     static InspectorDebuggerAgent* s_debuggerAgentOnBreakpoint;
     RefPtr<InspectorObject> m_breakProgramDetails;
+    bool m_javaScriptPauseScheduled;
 };
 
 } // namespace WebCore

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list