DevTools: move instrumentation of v8 calls into V8ScriptRunner

Source/core/inspector/V8DebuggerAgent.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/inspector/V8DebuggerAgent.h"
 
 #include "bindings/core/v8/ScriptCallStackFactory.h"
 #include "bindings/core/v8/ScriptRegexp.h"
 #include "bindings/core/v8/ScriptValue.h"
@@ skipping 1347 matching lines @@
 {
     m_asyncCallTrackingListeners.add(listener);
 }
 
 void V8DebuggerAgent::removeAsyncCallTrackingListener(AsyncCallTrackingListener* listener)
 {
     ASSERT(m_asyncCallTrackingListeners.contains(listener));
     m_asyncCallTrackingListeners.remove(listener);
 }
 
 void V8DebuggerAgent::willCallFunction(int scriptId)

[dgozman, 2015/08/29 00:04:39]

willExecuteScript

[yurys, 2015/08/29 00:14:24]

Done.

 {
     changeJavaScriptRecursionLevel(+1);
     // Fast return.
     if (m_scheduledDebuggerStep != StepInto)
         return;
     // Skip unknown scripts (e.g. InjectedScript).
     if (!m_scripts.contains(String::number(scriptId)))
         return;
     schedulePauseOnNextStatementIfSteppingInto();
 }
 
 void V8DebuggerAgent::didCallFunction()
 {
     changeJavaScriptRecursionLevel(-1);
 }
 
-void V8DebuggerAgent::willEvaluateScript()
-{
-    changeJavaScriptRecursionLevel(+1);
-    schedulePauseOnNextStatementIfSteppingInto();
-}
-
-void V8DebuggerAgent::didEvaluateScript()
-{
-    changeJavaScriptRecursionLevel(-1);
-}
-
 void V8DebuggerAgent::changeJavaScriptRecursionLevel(int step)
 {
     if (m_javaScriptPauseScheduled && !m_skipAllPauses && !isPaused()) {
         // Do not ever loose user's pause request until we have actually paused.
         debugger().setPauseOnNextStatement(true);
     }
     if (m_scheduledDebuggerStep == StepOut) {
         m_recursionLevelForStepOut += step;
         if (!m_recursionLevelForStepOut) {
             // When StepOut crosses a task boundary (i.e. js -> blink_c++) from where it was requested,
@@ skipping 298 matching lines @@
     RefPtr<ExceptionDetails> exceptionDetails = ExceptionDetails::create().setText(toCoreStringWithUndefinedOrNullCheck(message->Get()));
     exceptionDetails->setLine(message->GetLineNumber());
     exceptionDetails->setColumn(message->GetStartColumn());
     v8::Local<v8::StackTrace> messageStackTrace = message->GetStackTrace();
     if (!messageStackTrace.IsEmpty() && messageStackTrace->GetFrameCount() > 0)
         exceptionDetails->setStackTrace(createScriptCallStack(isolate, messageStackTrace, messageStackTrace->GetFrameCount())->buildInspectorArray());
     return exceptionDetails.release();
 }
 
 } // namespace blink