Web Inspector: Remove always false case in Debugger
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Sep 2016 23:31:37 +0000 (23:31 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Sep 2016 23:31:37 +0000 (23:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=161717

Patch by Joseph Pecoraro <pecoraro@apple.com> 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.

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/debugger/Debugger.cpp

index f754827..9f978e0 100644 (file)
@@ -1,3 +1,15 @@
+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>
 
         Unreviewed, build fix after r205569
index 577c6f2..7f3e31e 100644 (file)
@@ -757,8 +757,6 @@ void Debugger::didExecuteProgram(CallFrame* callFrame)
     if (m_currentCallFrame == m_pauseOnCallFrame) {
         VMEntryFrame* topVMEntryFrame = m_vm.topVMEntryFrame;
         m_pauseOnCallFrame = m_currentCallFrame->callerFrame(topVMEntryFrame);
-        if (!m_currentCallFrame)
-            return;
     }
     VMEntryFrame* topVMEntryFrame = m_vm.topVMEntryFrame;
     m_currentCallFrame = m_currentCallFrame->callerFrame(topVMEntryFrame);