[debugger] tuned StepNext and StepOut at return position

src/inspector/v8-debugger-agent-impl.cc

Index: src/inspector/v8-debugger-agent-impl.cc
diff --git a/src/inspector/v8-debugger-agent-impl.cc b/src/inspector/v8-debugger-agent-impl.cc
index 6648696593259e72eb725c292d43d5de4ec48342..d38e8134c86f8e58940983298a1db6ce65da48ca 100644
--- a/src/inspector/v8-debugger-agent-impl.cc
+++ b/src/inspector/v8-debugger-agent-impl.cc
@@ -731,9 +731,9 @@ Response V8DebuggerAgentImpl::resume() {
 Response V8DebuggerAgentImpl::stepOver() {
   if (!isPaused()) return Response::Error(kDebuggerNotPaused);
   // StepOver at function return point should fallback to StepInto.
-  JavaScriptCallFrame* frame =
-      !m_pausedCallFrames.empty() ? m_pausedCallFrames[0].get() : nullptr;
-  if (frame && frame->isAtReturn()) return stepInto();
+  // JavaScriptCallFrame* frame =

[kozy, 2017/03/16 07:19:00]

I can remove it in this CL since we use to step between embedder call, I'll be
able to remove it in followup:
https://codereview.chromium.org/2748503002/

+  //     !m_pausedCallFrames.empty() ? m_pausedCallFrames[0].get() : nullptr;
+  // if (frame && frame->isAtReturn()) return stepInto();
   m_scheduledDebuggerStep = StepOver;
   m_session->releaseObjectGroup(kBacktraceObjectGroup);
   m_debugger->stepOverStatement();