Implement rewind: drop one or more frames from the debugger.

runtime/vm/service.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/service.h"
 
 #include "include/dart_api.h"
 #include "include/dart_native_api.h"
 #include "platform/globals.h"
 
@@ skipping 2970 matching lines @@
   int64_t time_origin_micros =
       Int64Parameter::Parse(js->LookupParam("timeOriginMicros"));
   int64_t time_extent_micros =
       Int64Parameter::Parse(js->LookupParam("timeExtentMicros"));
   TimelineEventFilter filter(time_origin_micros, time_extent_micros);
   timeline_recorder->PrintJSON(js, &filter);
   return true;
 }
 
 
+static const char* const step_enum_names[] = {
+    "None", "Into", "Over", "Out", "Rewind", "OverAsyncSuspension", NULL,
+};
+
+
+static const Debugger::ResumeAction step_enum_values[] = {
+    Debugger::kContinue,   Debugger::kStepInto,
+    Debugger::kStepOver,   Debugger::kStepOut,
+    Debugger::kStepRewind, Debugger::kStepOverAsyncSuspension,
+    Debugger::kContinue,  // Default value
+};
+
+
 static const MethodParameter* resume_params[] = {
-    RUNNABLE_ISOLATE_PARAMETER, NULL,
+    RUNNABLE_ISOLATE_PARAMETER,
+    new EnumParameter("step", false, step_enum_names),
+    new UIntParameter("frameIndex", false), NULL,
 };
 
 
 static bool Resume(Thread* thread, JSONStream* js) {
   const char* step_param = js->LookupParam("step");
+  Debugger::ResumeAction step = Debugger::kContinue;
+  if (step_param != NULL) {
+    step = EnumMapper(step_param, step_enum_names, step_enum_values);
+  }
+#if defined(TARGET_ARCH_DBC)
+  if (step == Debugger::kStepRewind) {
+    js->PrintError(kCannotResume,
+                   "Rewind not yet implemented on this architecture");
+    return true;
+  }
+#endif
+  intptr_t frame_index = 1;
+  const char* frame_index_param = js->LookupParam("frameIndex");
+  if (frame_index_param != NULL) {
+    if (step != Debugger::kStepRewind) {
+      // Only rewind supports the frameIndex parameter.
+      js->PrintError(
+          kInvalidParams,
+          "%s: the 'frameIndex' parameter can only be used when rewinding",
+          js->method());
+      return true;
+    }
+    frame_index = UIntParameter::Parse(js->LookupParam("frameIndex"));
+  }
   Isolate* isolate = thread->isolate();
   if (isolate->message_handler()->is_paused_on_start()) {
     // If the user is issuing a 'Over' or an 'Out' step, that is the
     // same as a regular resume request.
-    if ((step_param != NULL) && (strcmp(step_param, "Into") == 0)) {
+    if (step == Debugger::kStepInto) {
       isolate->debugger()->EnterSingleStepMode();
     }
     isolate->message_handler()->set_should_pause_on_start(false);
     isolate->SetResumeRequest();
     if (Service::debug_stream.enabled()) {
       ServiceEvent event(isolate, ServiceEvent::kResume);
       Service::HandleEvent(&event);
     }
     PrintSuccess(js);
     return true;
   }
   if (isolate->message_handler()->should_pause_on_start()) {
     isolate->message_handler()->set_should_pause_on_start(false);
     isolate->SetResumeRequest();
     if (Service::debug_stream.enabled()) {
       ServiceEvent event(isolate, ServiceEvent::kResume);
       Service::HandleEvent(&event);
     }
     PrintSuccess(js);
     return true;
   }
   if (isolate->message_handler()->is_paused_on_exit()) {
     isolate->message_handler()->set_should_pause_on_exit(false);
     isolate->SetResumeRequest();
     // We don't send a resume event because we will be exiting.
     PrintSuccess(js);
     return true;
   }
-  if (isolate->debugger()->PauseEvent() != NULL) {
-    if (step_param != NULL) {
-      if (strcmp(step_param, "Into") == 0) {
-        isolate->debugger()->SetSingleStep();
-      } else if (strcmp(step_param, "Over") == 0) {
-        isolate->debugger()->SetStepOver();
-      } else if (strcmp(step_param, "Out") == 0) {
-        isolate->debugger()->SetStepOut();
-      } else if (strcmp(step_param, "OverAsyncSuspension") == 0) {
-        if (!isolate->debugger()->SetupStepOverAsyncSuspension()) {
-          js->PrintError(kInvalidParams,
-                         "Isolate must be paused at an async suspension point");
-          return true;
-        }
-      } else {
-        PrintInvalidParamError(js, "step");
-        return true;
-      }
-    }
-    isolate->SetResumeRequest();
-    PrintSuccess(js);
+  if (isolate->debugger()->PauseEvent() == NULL) {
+    js->PrintError(kIsolateMustBePaused, NULL);
     return true;
   }
 
-  js->PrintError(kIsolateMustBePaused, NULL);
+  const char* error = NULL;
+  if (!isolate->debugger()->SetResumeAction(step, frame_index, &error)) {
+    js->PrintError(kCannotResume, error);
+    return true;
+  }
+  isolate->SetResumeRequest();
+  PrintSuccess(js);
   return true;
 }
 
 
 static const MethodParameter* pause_params[] = {
     RUNNABLE_ISOLATE_PARAMETER, NULL,
 };
 
 
 static bool Pause(Thread* thread, JSONStream* js) {
@@ skipping 1015 matching lines @@
     if (strcmp(method_name, method.name) == 0) {
       return &method;
     }
   }
   return NULL;
 }
 
 #endif  // !PRODUCT
 
 }  // namespace dart