WorkerLoaderProxy: stop accessing cross-thread execution context

third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/workers/ThreadedMessagingProxyBase.h"
 
 #include "bindings/core/v8/SourceLocation.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContextTask.h"
 #include "core/frame/Deprecation.h"
@@ skipping 54 matching lines @@
     std::unique_ptr<WTF::CrossThreadClosure> task) {
   if (m_askedToTerminate)
     return;
 
   DCHECK(m_workerThread);
   m_workerThread->postTask(location, std::move(task));
 }
 
 void ThreadedMessagingProxyBase::postTaskToLoader(
     const WebTraceLocation& location,
-    std::unique_ptr<ExecutionContextTask> task) {
-  DCHECK(getExecutionContext()->isDocument());
-  getParentFrameTaskRunners()
-      ->get(TaskType::Networking)
-      ->postTask(BLINK_FROM_HERE,
-                 crossThreadBind(
-                     &ExecutionContextTask::performTaskIfContextIsValid,
-                     WTF::passed(std::move(task)),
-                     wrapCrossThreadWeakPersistent(getExecutionContext())));
+    std::unique_ptr<WTF::CrossThreadClosure> task) {
+  m_parentFrameTaskRunners->get(TaskType::Networking)
+      ->postTask(BLINK_FROM_HERE, std::move(task));
+}
+
+ExecutionContext* ThreadedMessagingProxyBase::getLoaderExecutionContext() {
+  DCHECK(isParentContextThread());
+  return getExecutionContext();
 }
 
 void ThreadedMessagingProxyBase::countFeature(UseCounter::Feature feature) {
   DCHECK(isParentContextThread());
   UseCounter::count(m_executionContext, feature);
 }
 
 void ThreadedMessagingProxyBase::countDeprecation(UseCounter::Feature feature) {
   DCHECK(isParentContextThread());
   Deprecation::countDeprecation(m_executionContext, feature);
@@ skipping 67 matching lines @@
 void ThreadedMessagingProxyBase::postMessageToPageInspector(
     const String& message) {
   DCHECK(isParentContextThread());
   if (m_workerInspectorProxy)
     m_workerInspectorProxy->dispatchMessageFromWorker(message);
 }
 
 bool ThreadedMessagingProxyBase::isParentContextThread() const {
   // TODO(nhiroki): Nested worker is not supported yet, so the parent context
   // thread should be equal to the main thread (http://crbug.com/31666).
-  DCHECK(getExecutionContext()->isDocument());
+  DCHECK(m_executionContext->isDocument());
   return isMainThread();
 }
 
 }  // namespace blink