Use GetDebuggedContext instead of GetCallingContext in v8_inspector

third_party/WebKit/Source/platform/v8_inspector/InspectedContext.cpp

Index: third_party/WebKit/Source/platform/v8_inspector/InspectedContext.cpp
diff --git a/third_party/WebKit/Source/platform/v8_inspector/InspectedContext.cpp b/third_party/WebKit/Source/platform/v8_inspector/InspectedContext.cpp
index 5b52e015a5436383989cf30a72017f8e324ca449..047c8a008a75e253b6123227359cf7d513a9e7ac 100644
--- a/third_party/WebKit/Source/platform/v8_inspector/InspectedContext.cpp
+++ b/third_party/WebKit/Source/platform/v8_inspector/InspectedContext.cpp
@@ -74,8 +74,10 @@ void InspectedContext::createInjectedScript()
     DCHECK(!m_injectedScript);
     v8::HandleScope handles(isolate());
     v8::Local<v8::Context> localContext = context();
-    v8::Local<v8::Context> callingContext = isolate()->GetCallingContext();
-    if (!callingContext.IsEmpty() && !m_debugger->client()->callingContextCanAccessContext(callingContext, localContext))
+    v8::Local<v8::Context> callingContext;

[haraken, 2016/06/07 14:07:30]

Ditto.

+    if (!v8::Debug::GetDebuggedContext(isolate()).ToLocal(&callingContext))

[pfeldman, 2016/06/07 14:39:29]

We should not need calling context here - injected script is created in the
controlled code path, so we should be able to enter the context.

[jochen (gone - plz use gerrit), 2016/06/07 14:52:13]

so just delete this check?

[kozy, 2016/06/08 19:28:23]

I'm worried here about V8DebuggerAgent::currentCallFrames. In this call we
create related InjectedScript for each call frame in current stack on pause and
callingContext is top callFrame, targetContext is frame context. But if we are
able to call function from target context then it will be safe to create
InjectedScript in target context. If this case is safe then we can remove this
check.

[dgozman, 2016/06/09 08:53:45]

I think we'd better keep it for this call:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/v8_in...

This access checks are sanity checks anyway, since they originate from native
calls, and we only check the calling context, not the object being wrapped
"belonging" to the InjectedScript's context.

+        return;
+    if (!m_debugger->client()->callingContextCanAccessContext(callingContext, localContext))
         return;
     m_injectedScript = InjectedScript::create(this);
 }