Fix DevTools support of worklets

third_party/WebKit/Source/core/inspector/MainThreadDebugger.cpp

 /*
  * Copyright (c) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 169 matching lines @@
 void MainThreadDebugger::muteConsole()
 {
     FrameConsole::mute();
 }
 
 void MainThreadDebugger::unmuteConsole()
 {
     FrameConsole::unmute();
 }
 
 bool MainThreadDebugger::callingContextCanAccessContext(v8::Local<v8::Context> calling, v8::Local<v8::Context> target)

[haraken, 2016/06/29 00:23:35]

Not directly related to the CL, but is there any reason you have to use a
calling context here? We're deprecating a calling context (because it's broken
in many ways) and replacing it with a current context. Would it be possible to
use a current context instead?

[Gleb Lanbin, 2016/06/29 17:07:13]

please see the reply from dgozman@ above in this thread

 {
-    ExecutionContext* executionContext = toExecutionContext(target);
-    ASSERT(executionContext);
-
-    if (executionContext->isMainThreadWorkletGlobalScope()) {
-        MainThreadWorkletGlobalScope* globalScope = toMainThreadWorkletGlobalScope(executionContext);
-        return globalScope && BindingSecurity::shouldAllowAccessTo(m_isolate, toLocalDOMWindow(toDOMWindow(calling)), globalScope, DoNotReportSecurityError);
-    }
-
-    DOMWindow* window = toDOMWindow(target);
-    return window && BindingSecurity::shouldAllowAccessTo(m_isolate, toLocalDOMWindow(toDOMWindow(calling)), window, DoNotReportSecurityError);
+    return BindingSecurity::shouldAllowAccessTo(m_isolate, calling, target, DoNotReportSecurityError);
 }
 
 int MainThreadDebugger::ensureDefaultContextInGroup(int contextGroupId)
 {
     LocalFrame* frame = WeakIdentifierMap<LocalFrame>::lookup(contextGroupId);
     if (!frame)
         return 0;
     ScriptState* scriptState = ScriptState::forMainWorld(frame);
     if (!scriptState)
         return 0;
@@ skipping 127 matching lines @@
                 return;
             if (!nodes->Set(context, index++, toV8(node, info.Holder(), info.GetIsolate())).FromMaybe(false))
                 return;
         }
         info.GetReturnValue().Set(nodes);
     }
     exceptionState.throwIfNeeded();
 }
 
 } // namespace blink