[debug] remove StepFrame

src/debug/debug.cc

Index: src/debug/debug.cc
diff --git a/src/debug/debug.cc b/src/debug/debug.cc
index 31949bb5530b02743966d0e55379932f14719933..63389868316ce1b0cca5a12bb067516daa2d5142 100644
--- a/src/debug/debug.cc
+++ b/src/debug/debug.cc
@@ -561,9 +561,6 @@ void Debug::Break(JavaScriptFrame* frame) {
                        summary.SourceStatementPosition();
       break;
     }
-    case StepFrame:
-      step_break = current_fp != last_fp;
-      break;
   }
 
   // Clear all current stepping setup.
@@ -1082,12 +1079,6 @@ void Debug::PrepareStep(StepAction step_action) {
       // TODO(clemensh): Implement stepping from JS into WASM.
       FloodWithOneShot(function);
       break;
-    case StepFrame:
-      // TODO(clemensh): Implement stepping from JS into WASM or vice versa.
-      // No point in setting one-shot breaks at places where we are not about
-      // to leave the current frame.
-      FloodWithOneShot(function, CALLS_AND_RETURNS);

[kozy, 2017/01/25 20:21:05]

We could remove CALLS_AND_RETURNS as well. But as I understood your're going to
use it for break_on_function_entry.

-      break;
   }
 }
 
@@ -2153,9 +2144,8 @@ void Debug::UpdateState() {
 }
 
 void Debug::UpdateHookOnFunctionCall() {
-  STATIC_ASSERT(StepFrame > StepIn);
-  STATIC_ASSERT(LastStepAction == StepFrame);
-  hook_on_function_call_ = thread_local_.last_step_action_ >= StepIn ||
+  STATIC_ASSERT(LastStepAction == StepIn);
+  hook_on_function_call_ = thread_local_.last_step_action_ == StepIn ||
                            isolate_->needs_side_effect_check();
 }