Revert of Remove V8RecrusionScope, cleanup call sites.

Revert of Remove V8RecrusionScope, cleanup call sites. (patchset #8 id:140001 of https://codereview.chromium.org/1769273004/ )

Reason for revert:
Speculative revert, this patch most likely to cause browser_tests failures for Linux (https://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/26678) Windows (https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/46875) and Mac

Original issue's description:
> Remove V8RecrusionScope, cleanup call sites.
> 
> - blink-side usages migrated to v8::MicrotasksScope;
> - clients of WebScopedMicrotaskSuppression migrated to v8::MicrotasksScope;
> - moved Microtask.{h,cpp} to bindings/core/v8;
> - removed extra wrappers from V8DebuggerClient;
> - fixed a couple of inspector tests which relied on trace event not emitted from debugger anymore.
> 
> BUG=585949
> 
> Committed: https://crrev.com/133a1579d2a087acdcdd4e9247062edb0bd1e5a3
> Cr-Commit-Position: refs/heads/master@{#381103}

TBR=jochen@chromium.org,pfeldman@chromium.org,dgozman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=585949

[dvadym, 2016-03-15 14:07:36]

Created Revert of Remove V8RecrusionScope, cleanup call sites.

[commit-bot: I haz the power, 2016-03-15 14:07:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1804043002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1804043002/1

[commit-bot: I haz the power, 2016-03-15 14:08:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-15 14:08:47]

Failed to apply patch for
third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp:237
  Falling back to three-way merge...
  Applied patch to
'third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp' with
conflicts.
  U third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp

Patch:       third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
Index: third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
diff --git a/third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
b/third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
index
dc59c7a50001c7872550ed84d7e709184af1c1ef..ba26c811cb0df93f560a9a13aa1b71c605dc306c
100644
--- a/third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
+++ b/third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
@@ -36,7 +36,6 @@
 #include "platform/v8_inspector/InjectedScriptHost.h"
 #include "platform/v8_inspector/InjectedScriptManager.h"
 #include "platform/v8_inspector/RemoteObjectId.h"
-#include "platform/v8_inspector/V8DebuggerImpl.h"
 #include "platform/v8_inspector/V8FunctionCall.h"
 #include "platform/v8_inspector/V8StringUtil.h"
 #include "platform/v8_inspector/public/V8Debugger.h"
@@ -116,11 +115,12 @@
     data.GetParameter()->dispose();
 }
 
-InjectedScript::InjectedScript(InjectedScriptManager* manager,
v8::Local<v8::Context> context, v8::Local<v8::Object> object,
PassOwnPtr<InjectedScriptNative> injectedScriptNative, int contextId)
+InjectedScript::InjectedScript(InjectedScriptManager* manager,
v8::Local<v8::Context> context, v8::Local<v8::Object> object, V8DebuggerClient*
client, PassOwnPtr<InjectedScriptNative> injectedScriptNative, int contextId)
     : m_manager(manager)
     , m_isolate(context->GetIsolate())
     , m_context(m_isolate, context)
     , m_value(m_isolate, object)
+    , m_client(client)
     , m_native(injectedScriptNative)
     , m_contextId(contextId)
 {
@@ -134,7 +134,7 @@
 void InjectedScript::evaluate(ErrorString* errorString, const String16&
expression, const String16& objectGroup, bool includeCommandLineAPI, bool
returnByValue, bool generatePreview, OwnPtr<protocol::Runtime::RemoteObject>*
result, Maybe<bool>* wasThrown, Maybe<protocol::Runtime::ExceptionDetails>*
exceptionDetails)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"evaluate");
+    V8FunctionCall function(m_client, context(), v8Value(), "evaluate");
     function.appendArgument(expression);
     function.appendArgument(objectGroup);
     function.appendArgument(includeCommandLineAPI);
@@ -146,7 +146,7 @@
 void InjectedScript::callFunctionOn(ErrorString* errorString, const String16&
objectId, const String16& expression, const String16& arguments, bool
returnByValue, bool generatePreview, OwnPtr<protocol::Runtime::RemoteObject>*
result, Maybe<bool>* wasThrown)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"callFunctionOn");
+    V8FunctionCall function(m_client, context(), v8Value(), "callFunctionOn");
     function.appendArgument(objectId);
     function.appendArgument(expression);
     function.appendArgument(arguments);
@@ -158,7 +158,7 @@
 void InjectedScript::evaluateOnCallFrame(ErrorString* errorString,
v8::Local<v8::Object> callFrames, const String16& callFrameId, const String16&
expression, const String16& objectGroup, bool includeCommandLineAPI, bool
returnByValue, bool generatePreview, OwnPtr<RemoteObject>* result, Maybe<bool>*
wasThrown, Maybe<protocol::Runtime::ExceptionDetails>* exceptionDetails)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"evaluateOnCallFrame");
+    V8FunctionCall function(m_client, context(), v8Value(),
"evaluateOnCallFrame");
     function.appendArgument(callFrames);
     function.appendArgument(callFrameId);
     function.appendArgument(expression);
@@ -172,7 +172,7 @@
 void InjectedScript::restartFrame(ErrorString* errorString,
v8::Local<v8::Object> callFrames, const String16& callFrameId)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"restartFrame");
+    V8FunctionCall function(m_client, context(), v8Value(), "restartFrame");
     function.appendArgument(callFrames);
     function.appendArgument(callFrameId);
     OwnPtr<protocol::Value> resultValue = makeCall(function);
@@ -197,7 +197,7 @@
     const String16& newValueStr)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"setVariableValue");
+    V8FunctionCall function(m_client, context(), v8Value(),
"setVariableValue");
     if (callFrameIdOpt.isJust()) {
         function.appendArgument(callFrames);
         function.appendArgument(callFrameIdOpt.fromJust());
@@ -227,7 +227,7 @@
 void InjectedScript::getFunctionDetails(ErrorString* errorString, const
String16& functionId, OwnPtr<FunctionDetails>* result)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"getFunctionDetails");
+    V8FunctionCall function(m_client, context(), v8Value(),
"getFunctionDetails");
     function.appendArgument(functionId);
     OwnPtr<protocol::Value> resultValue = makeCall(function);
     protocol::ErrorSupport errors(errorString);
@@ -237,7 +237,7 @@
 void InjectedScript::getGeneratorObjectDetails(ErrorString* errorString, const
String16& objectId, OwnPtr<GeneratorObjectDetails>* result)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"getGeneratorObjectDetails");
+    V8FunctionCall function(m_client, context(), v8Value(),
"getGeneratorObjectDetails");
     function.appendArgument(objectId);
     OwnPtr<protocol::Value> resultValue = makeCall(function);
     protocol::ErrorSupport errors(errorString);
@@ -247,7 +247,7 @@
 void InjectedScript::getCollectionEntries(ErrorString* errorString, const
String16& objectId, OwnPtr<Array<CollectionEntry>>* result)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"getCollectionEntries");
+    V8FunctionCall function(m_client, context(), v8Value(),
"getCollectionEntries");
     function.appendArgument(objectId);
     OwnPtr<protocol::Value> resultValue = makeCall(function);
     protocol::ErrorSupport errors(errorString);
@@ -257,7 +257,7 @@
 void InjectedScript::getProperties(ErrorString* errorString, const String16&
objectId, bool ownProperties, bool accessorPropertiesOnly, bool generatePreview,
OwnPtr<Array<PropertyDescriptor>>* properties,
Maybe<protocol::Runtime::ExceptionDetails>* exceptionDetails)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"getProperties");
+    V8FunctionCall function(m_client, context(), v8Value(), "getProperties");
     function.appendArgument(objectId);
     function.appendArgument(ownProperties);
     function.appendArgument(accessorPropertiesOnly);
@@ -276,7 +276,7 @@
 void InjectedScript::getInternalProperties(ErrorString* errorString, const
String16& objectId, Maybe<Array<InternalPropertyDescriptor>>* properties,
Maybe<protocol::Runtime::ExceptionDetails>* exceptionDetails)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"getInternalProperties");
+    V8FunctionCall function(m_client, context(), v8Value(),
"getInternalProperties");
     function.appendArgument(objectId);
 
     OwnPtr<protocol::Value> result = makeCallWithExceptionDetails(function,
exceptionDetails);
@@ -307,14 +307,14 @@
     v8::Local<v8::Symbol> commandLineAPISymbolValue =
V8Debugger::commandLineAPISymbol(m_isolate);
     v8::Local<v8::Object> global = context()->Global();
     if (includeCommandLineAPI) {
-        V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"commandLineAPI");
+        V8FunctionCall function(m_client, context(), v8Value(),
"commandLineAPI");
         bool hadException = false;
         v8::Local<v8::Value> commandLineAPI = function.call(hadException,
false);
         if (!hadException)
             global->Set(commandLineAPISymbolValue, commandLineAPI);
     }
 
-    v8::MaybeLocal<v8::Value> maybeValue =
m_manager->debugger()->runCompiledScript(context(), script);
+    v8::MaybeLocal<v8::Value> maybeValue =
m_client->runCompiledScript(context(), script);
     if (includeCommandLineAPI)
         global->Delete(context(), commandLineAPISymbolValue);
 
@@ -324,7 +324,7 @@
 PassOwnPtr<Array<CallFrame>>
InjectedScript::wrapCallFrames(v8::Local<v8::Object> callFrames)
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"wrapCallFrames");
+    V8FunctionCall function(m_client, context(), v8Value(), "wrapCallFrames");
     function.appendArgument(callFrames);
     bool hadException = false;
     v8::Local<v8::Value> callFramesValue =
callFunctionWithEvalEnabled(function, hadException);
@@ -339,7 +339,7 @@
 PassOwnPtr<protocol::Runtime::RemoteObject>
InjectedScript::wrapObject(v8::Local<v8::Value> value, const String16&
groupName, bool generatePreview) const
 {
     v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(),
"wrapObject");
+    V8FunctionCall function(m_client, context(), v8Value(), "wrapObject");
     function.appendArgument(value);
     function.appendArgument(groupName);
     function.appendArgument(canAccessInspectedWindow());
@@ -355,7 +355,7 @@
 PassOwnPtr<protocol::Runtime::RemoteObject>
InjectedScript::wrapTable(v8::Local<v8::Value> table, v8::Loca…
(message too large)