[inspector] Add some context scopes to inspector code

src/inspector/injected-script.cc

Index: src/inspector/injected-script.cc
diff --git a/src/inspector/injected-script.cc b/src/inspector/injected-script.cc
index b4183a735a250f8d0493050df0401404626b51a1..183fa446b35d9b821b0ba24cb0776cb16b6a8bd1 100644
--- a/src/inspector/injected-script.cc
+++ b/src/inspector/injected-script.cc
@@ -175,6 +175,7 @@ Response InjectedScript::wrapObject(
   v8::HandleScope handles(m_context->isolate());
   v8::Local<v8::Value> wrappedObject;
   v8::Local<v8::Context> context = m_context->context();
+  v8::Context::Scope contextScope(context);

[dgozman, 2017/04/26 15:52:49]

I don't think this is needed. It calls into wrapValue, which calls into
V8FunctionCall.

Overall, I think it only makes sense to enter context before accessing V8 api.
WDYT?

[Jens Widell, 2017/04/26 15:57:13]

Indeed. I now wonder how come I ended up adding this one. I'll test without it
to make sure there wasn't a reason, then drop.
Yes, absolutely. Only reason to cover more code than just V8 API usage would be
if it avoided adding context scopes all over the place, by putting them in
logical central locations. But that's not at all the case here.

   Response response = wrapValue(value, groupName, forceValueType,
                                 generatePreview, &wrappedObject);
   if (!response.isSuccess()) return response;