[BugFix][Debugger] Fix Debugger.skipAllPauses issue.

Synchronize the values of breakpoints_is_active_before and exception_breakpoint_before with breakpoints_is_active and exception_breakpoint, instead of assigning them only when receiving Debugger.skipAllPauses messages with the skip parameter set to true. Otherwise, if only Debugger.skipAllPauses messages with the skip parameter set to false are received, it will cause a state error.

Author: ci_lynx

src/inspector/debugger/debugger.cc

@@ -1542,7 +1542,7 @@ void HandleEnable(DebuggerParams *debugger_options) {
 
     bool is_already_enabled = false;
     bool is_paused = false;
-    info->breakpoints_is_active = 1;
+    info->breakpoints_is_active_before = info->breakpoints_is_active = 1;
     if (view_id != -1) {
       // get if session is enabled and if session is paused
       GetSessionState(ctx, view_id, &is_already_enabled, &is_paused);
@@ -1590,8 +1590,6 @@ void HandleSkipAllPauses(DebuggerParams *debugger_options) {
     int32_t is_skip = LEPUS_VALUE_GET_BOOL(params_skip);
     if (!ctx->rt->gc_enable) LEPUS_FreeValue(ctx, params);
     if (is_skip) {
-      info->breakpoints_is_active_before = info->breakpoints_is_active;
-      info->exception_breakpoint_before = info->exception_breakpoint;
       info->breakpoints_is_active = 0;
       info->exception_breakpoint = 0;
     } else {
@@ -1890,9 +1888,9 @@ void HandleSetPauseOnExceptions(DebuggerParams *debugger_options) {
     }
     if (state) {
       if (strcmp("uncaught", state) == 0 || strcmp("all", state) == 0) {
-        info->exception_breakpoint = 1;
+        info->exception_breakpoint_before = info->exception_breakpoint = 1;
       } else if (strcmp("none", state) == 0) {
-        info->exception_breakpoint = 0;
+        info->exception_breakpoint_before = info->exception_breakpoint = 0;
       }
       if (!ctx->rt->gc_enable) LEPUS_FreeCString(ctx, state);
 

src/inspector/debugger/debugger_breakpoint.cc

@@ -703,7 +703,8 @@ void HandleSetBreakpointActive(DebuggerParams *debugger_options) {
   int32_t is_active = LEPUS_VALUE_GET_BOOL(active);
   if (!ctx->rt->gc_enable) LEPUS_FreeValue(ctx, params);
 
-  info->breakpoints_is_active = is_active ? 1 : 0;
+  info->breakpoints_is_active_before = info->breakpoints_is_active =
+      is_active ? 1 : 0;
 
   LEPUSValue result = LEPUS_NewObject(ctx);
   if (LEPUS_IsException(result)) {
@@ -981,7 +982,7 @@ void HandleContinueToLocation(DebuggerParams *debugger_options) {
   const char *url = GetScriptURLByScriptId(ctx, script_id);
 
   // set brekapoint active is 1
-  info->breakpoints_is_active = 1;
+  info->breakpoints_is_active_before = info->breakpoints_is_active = 1;
 
   // add this new breakpoint to the info->breakpoints, set specific_condition
   // true