Fix bug that debugger doesn't stop at the final line in Chrome when i…

…t proceeds with 'StepOver' The reason is that order of appending debug commands is incorrect
ruby · Dec 15, 2021 · dbbee39 · dbbee39
1 parent 1f18dc5
commit dbbee39
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/lib/debug/server_cdp.rb b/lib/debug/server_cdp.rb
@@ -314,7 +314,6 @@ def process
           send_response req
           send_event 'Debugger.resumed'
         when 'Debugger.stepOver'
-          @q_msg << req
           begin
             @session.check_postmortem
             @q_msg << 'n'
@@ -324,9 +323,10 @@ def process
             send_fail_response req,
                               code: INVALID_REQUEST,
                               message: "'stepOver' is not supported while postmortem mode"
+          ensure
+            @q_msg << req
           end
         when 'Debugger.stepInto'
-          @q_msg << req
           begin
             @session.check_postmortem
             @q_msg << 's'
@@ -336,9 +336,10 @@ def process
             send_fail_response req,
                               code: INVALID_REQUEST,
                               message: "'stepInto' is not supported while postmortem mode"
+          ensure
+            @q_msg << req
           end
         when 'Debugger.stepOut'
-          @q_msg << req
           begin
             @session.check_postmortem
             @q_msg << 'fin'
@@ -348,6 +349,8 @@ def process
             send_fail_response req,
                               code: INVALID_REQUEST,
                               message: "'stepOut' is not supported while postmortem mode"
+          ensure
+            @q_msg << req
           end
         when 'Debugger.setSkipAllPauses'
           skip = req.dig('params', 'skip')