Use GetDebuggedContext instead of GetCallingContext in v8_inspector

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 ec1828a24daa069536b54b7d3d653779ad704e78..9a23ea68e53e02517f73c625868d4a0d4eabf40a 100644
--- a/third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
+++ b/third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp
@@ -273,8 +273,8 @@ void InjectedScript::setCustomObjectFormatterEnabled(bool enabled)
 
 bool InjectedScript::canAccessInspectedWindow() const
 {
-    v8::Local<v8::Context> callingContext = m_context->isolate()->GetCallingContext();
-    if (callingContext.IsEmpty())
+    v8::Local<v8::Context> callingContext;

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

callingContext => debuggerContext ?

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

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

Note that GetDebuggedContext says "while in debug context", which is not always
the case here - we may wrap some objects (e.g. DOM nodes) when not entered into
debugger context.

I don't see how implementation of GetDebuggedContext uses that fact though.
Should we change the comment?

         return true;
     return m_context->debugger()->client()->callingContextCanAccessContext(callingContext, m_context->context());
 }