Changeset 205571 in webkit

Timestamp:

Sep 7, 2016 4:31:37 PM (8 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Remove always false case in Debugger
​https://bugs.webkit.org/show_bug.cgi?id=161717

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2016-09-07
Reviewed by Brian Burg.

• debugger/Debugger.cpp:

(JSC::Debugger::didExecuteProgram):
We would have earlier returned a few statements ago if this case was true.
And we would have crashed in the previous statement if this case was true.

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• debugger/Debugger.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2016-09-07  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Remove always false case in Debugger
+        https://bugs.webkit.org/show_bug.cgi?id=161717
+
+        Reviewed by Brian Burg.
+
+        * debugger/Debugger.cpp:
+        (JSC::Debugger::didExecuteProgram):
+        We would have earlier returned a few statements ago if this case was true.
+        And we would have crashed in the previous statement if this case was true.
+
 2016-09-07  Yusuke Suzuki  <utatane.tea@gmail.com>
 

trunk/Source/JavaScriptCore/debugger/Debugger.cpp

@@ -758 +758 @@
         VMEntryFrame* topVMEntryFrame = m_vm.topVMEntryFrame;
         m_pauseOnCallFrame = m_currentCallFrame->callerFrame(topVMEntryFrame);
-        if (!m_currentCallFrame)
-            return;
     }
     VMEntryFrame* topVMEntryFrame = m_vm.topVMEntryFrame;