Worker: Introduce per-global-scope task scheduler

third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp

 /*
  * Copyright (C) 2013 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 16 matching lines @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "web/WebEmbeddedWorkerImpl.h"
 
 #include <memory>
 #include "bindings/core/v8/SourceLocation.h"
 #include "core/dom/Document.h"
 #include "core/dom/SecurityContext.h"
+#include "core/dom/TaskRunnerHelper.h"
 #include "core/frame/csp/ContentSecurityPolicy.h"
 #include "core/inspector/ConsoleMessage.h"
 #include "core/loader/FrameLoadRequest.h"
 #include "core/loader/ThreadableLoadingContext.h"
 #include "core/probe/CoreProbes.h"
 #include "core/workers/ParentFrameTaskRunners.h"
 #include "core/workers/WorkerClients.h"
 #include "core/workers/WorkerGlobalScope.h"
 #include "core/workers/WorkerInspectorProxy.h"
 #include "core/workers/WorkerLoaderProxy.h"
@@ skipping 220 matching lines @@
     std::unique_ptr<WTF::CrossThreadClosure> task) {
   main_thread_task_runners_->Get(TaskType::kNetworking)
       ->PostTask(BLINK_FROM_HERE, std::move(task));
 }
 
 void WebEmbeddedWorkerImpl::PostTaskToWorkerGlobalScope(
     const WebTraceLocation& location,
     std::unique_ptr<WTF::CrossThreadClosure> task) {
   if (asked_to_terminate_ || !worker_thread_)
     return;
-  worker_thread_->PostTask(location, std::move(task));
+  TaskRunnerHelper::Get(TaskType::kNetworking, worker_thread_.get())

[haraken, 2017/04/17 14:49:52]

If we want to use kNetworking, I think we should rename
PostTaskToWorkerGlobalScope. PostTaskToWorkerGlobalScope sounds general and it's
not clear that networking-related tasks will be posted.

[kinuko, 2017/04/17 15:55:19]

(I believe we're deprecating / cleaning up this.  Also let me note that the
header file clearly says it's for WorkerLoaderProxy overrides)

[nhiroki, 2017/04/18 05:34:38]

Yes, these will be removed by a follow-up CL:
https://codereview.chromium.org/2811993007/

... so let me keep them as they are in this CL.

+      ->PostTask(location, std::move(task));
 }
 
 ThreadableLoadingContext* WebEmbeddedWorkerImpl::GetThreadableLoadingContext() {
   if (!loading_context_) {
     loading_context_ = ThreadableLoadingContext::Create(
         *main_frame_->GetFrame()->GetDocument());
   }
   return loading_context_;
 }
 
@@ skipping 188 matching lines @@
   loader_proxy_ = WorkerLoaderProxy::Create(this);
   worker_thread_ =
       ServiceWorkerThread::Create(loader_proxy_, *worker_global_scope_proxy_);
   worker_thread_->Start(std::move(startup_data),
                         main_thread_task_runners_.Get());
   worker_inspector_proxy_->WorkerThreadCreated(document, worker_thread_.get(),
                                                script_url);
 }
 
 }  // namespace blink