[inspector] migrate stepping related methods to debug-interface

src/api.cc

Index: src/api.cc
diff --git a/src/api.cc b/src/api.cc
index 508431a6f4b64e3ef085e0e067911bf27eece951..c1a1c01a2e9da03de3524c86d0e5b538237d9472 100644
--- a/src/api.cc
+++ b/src/api.cc
@@ -8814,6 +8814,24 @@ void DebugInterface::ChangeBreakOnException(Isolate* isolate,
                                                     type != NoBreakOnException);
 }
 
+void DebugInterface::PrepareStep(Isolate* v8_isolate, StepAction action) {
+  i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
+  ENTER_V8(isolate);
+  CHECK(isolate->debug()->CheckExecutionState(isolate->debug()->break_id()));

[Yang, 2016/10/19 04:38:56]

Can we introduce a new version of CheckExecutionState that does not take a break
id? The current one would just compare debug->break_id() with itself.

[kozy, 2016/10/20 04:58:28]

Done.

+  // Clear all current stepping setup.
+  isolate->debug()->ClearStepping();
+  // Prepare step.
+  isolate->debug()->PrepareStep(static_cast<i::StepAction>(action));
+}
+
+void DebugInterface::ClearStepping(Isolate* v8_isolate) {
+  i::Isolate* isolate = reinterpret_cast<i::Isolate*>(v8_isolate);
+  ENTER_V8(isolate);
+  CHECK(isolate->debug()->CheckExecutionState(isolate->debug()->break_id()));
+  // Clear all current stepping setup.
+  isolate->debug()->ClearStepping();
+}
+
 Local<String> CpuProfileNode::GetFunctionName() const {
   const i::ProfileNode* node = reinterpret_cast<const i::ProfileNode*>(this);
   i::Isolate* isolate = node->isolate();