Revert of Worker: Introduce per-global-scope task scheduler

Revert of Worker: Introduce per-global-scope task scheduler (patchset #22 id:450001 of https://codereview.chromium.org/2806623004/ )

Reason for revert:
Seems to have caused a crash in fast/workers/termination-early.html on Webkit Linux Trusty Leak -
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Trusty_Leak/4107/layout-test-results/results.html
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/4107

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Trusty_Leak/4106/layout-test-results/fast/workers/termination-early-crash-log.txt
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/4106

Original issue's description:
> Worker: Introduce per-global-scope task scheduler
> 
> This CL introduces WorkerGlobalScopeScheduler to make it simpler and saner to
> manage tasks on worker threads.
> 
> Before this CL, tasks on worker threads were always pushed into the thread's
> default task queue, they should be handled in different ways based on their
> task types though. For example, worker tasks shouldn't run after the global
> scope is closed, while thread control tasks should run even after that. To
> achieve it, worker tasks were always wrapped with WorkerThread's method to check
> the current thread state. This was so tricky.
> 
> After this CL, each global scope has its own task queues like per-frame task
> scheduler. Worker tasks are pushed into the task queues, control tasks are still
> pushed into the thread's default task queue though. Thanks to this separation,
> we can abort only worker tasks by simply unregistering the queues when a global
> scope is closed.
> 
> In addition, this CL unifies postTask interfaces into TaskRunnerHelper (issue
> 694914), and enables any threads to post a task to worker threads because of
> thread-safe refcounted WebTaskRunner.
> 
> BUG=670534, 694914, 710364
> 
> Review-Url: https://codereview.chromium.org/2806623004
> Cr-Commit-Position: refs/heads/master@{#465841}
> Committed: https://chromium.googlesource.com/chromium/src/+/d56a35fb25dc388a0506174b9e3fda67337313ce

TBR=kinuko@chromium.org,haraken@chromium.org,tzik@chromium.org,skyostil@chromium.org,altimin@chromium.org,nhiroki@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=670534, 694914, 710364
Review-Url: https://codereview.chromium.org/2831843002
Cr-Commit-Position: refs/heads/master@{#465892}
Committed: https://chromium.googlesource.com/chromium/src/+/ee6a4f06b0e0561618669ed56e5997af91108c3e

[aboxhall, 2017-04-20 03:22:26]

The CQ bit was checked by aboxhall@chromium.org

[aboxhall, 2017-04-20 03:22:26]

Created Revert of Worker: Introduce per-global-scope task scheduler

[commit-bot: I haz the power, 2017-04-20 03:22:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-20 03:24:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-20 03:24:03]

Failed to apply patch for
third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp:83
  error: repository lacks the necessary blob to fall back on 3-way merge.
  error: third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp: patch does not
apply

Patch:       third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp
Index: third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp
diff --git a/third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp
b/third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp
index
dc573a5973e0ff34a4ff4ca95b94235cd440c221..1a1cd884f8c88fa051d8d28d5eb2e60333289e23
100644
--- a/third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp
+++ b/third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp
@@ -7,8 +7,6 @@
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/frame/LocalFrame.h"
-#include "core/workers/WorkerOrWorkletGlobalScope.h"
-#include "core/workers/WorkerThread.h"
 #include "platform/WebFrameScheduler.h"
 #include "platform/WebTaskRunner.h"
 #include "public/platform/Platform.h"
@@ -66,15 +64,10 @@
 
 RefPtr<WebTaskRunner> TaskRunnerHelper::Get(
     TaskType type,
-    ExecutionContext* executionContext) {
-  if (!executionContext)
-    return Get(type, ToDocument(executionContext));
-  if (executionContext->IsDocument())
-    return Get(type, ToDocument(executionContext));
-  if (executionContext->IsWorkerOrWorkletGlobalScope())
-    return Get(type, ToWorkerOrWorkletGlobalScope(executionContext));
-  executionContext = nullptr;
-  return Get(type, ToDocument(executionContext));
+    ExecutionContext* execution_context) {
+  return Get(type, execution_context && execution_context->IsDocument()
+                       ? static_cast<Document*>(execution_context)
+                       : nullptr);
 }
 
 RefPtr<WebTaskRunner> TaskRunnerHelper::Get(TaskType type,
@@ -83,49 +76,4 @@
              script_state ? ExecutionContext::From(script_state) : nullptr);
 }
 
-RefPtr<WebTaskRunner> TaskRunnerHelper::Get(
-    TaskType type,
-    WorkerOrWorkletGlobalScope* global_scope) {
-  DCHECK(global_scope);
-  // TODO(nhiroki): Add |DCHECK(global_scope->IsContextThread())| here after
-  // https://crbug.com/694925 is fixed.
-  return Get(type, global_scope->GetThread());
-}
-
-RefPtr<WebTaskRunner> TaskRunnerHelper::Get(TaskType type,
-                                            WorkerThread* worker_thread) {
-  switch (type) {
-    case TaskType::kDOMManipulation:
-    case TaskType::kUserInteraction:
-    case TaskType::kNetworking:
-    case TaskType::kHistoryTraversal:
-    case TaskType::kEmbed:
-    case TaskType::kMediaElementEvent:
-    case TaskType::kCanvasBlobSerialization:
-    case TaskType::kMicrotask:
-    case TaskType::kTimer:
-    case TaskType::kRemoteEvent:
-    case TaskType::kWebSocket:
-    case TaskType::kPostedMessage:
-    case TaskType::kUnshippedPortMessage:
-    case TaskType::kFileReading:
-    case TaskType::kDatabaseAccess:
-    case TaskType::kPresentation:
-    case TaskType::kSensor:
-    case TaskType::kPerformanceTimeline:
-    case TaskType::kWebGL:
-    case TaskType::kMiscPlatformAPI:
-    case TaskType::kUnspecedTimer:
-    case TaskType::kUnspecedLoading:
-    case TaskType::kUnthrottled:
-      // UnthrottledTaskRunner is generally discouraged in future.
-      // TODO(nhiroki): Identify which tasks can be throttled / suspendable and
-      // move them into other task runners. See also comments in
-      // Get(LocalFrame). (https://crbug.com/670534)
-      return worker_thread->GetGlobalScopeScheduler()->UnthrottledTaskRunner();
-  }
-  NOTREACHED();
-  return nullptr;
-}
-
 }  // namespace blink

[nhiroki, 2017-04-20 03:45:18]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2017-04-20 03:45:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-20 03:46:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492659918016630, "parent_rev":
"c695cc71ebc3348aa9db521f7f4324502a0134e3", "commit_rev":
"ee6a4f06b0e0561618669ed56e5997af91108c3e"}

[commit-bot: I haz the power, 2017-04-20 03:47:02]

Description was changed from

==========
Revert of Worker: Introduce per-global-scope task scheduler (patchset #22
id:450001 of https://codereview.chromium.org/2806623004/ )

Reason for revert:
Seems to have caused a crash in fast/workers/termination-early.html on Webkit
Linux Trusty Leak -
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Tru...
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Tru...
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

Original issue's description:
> Worker: Introduce per-global-scope task scheduler
> 
> This CL introduces WorkerGlobalScopeScheduler to make it simpler and saner to
> manage tasks on worker threads.
> 
> Before this CL, tasks on worker threads were always pushed into the thread's
> default task queue, they should be handled in different ways based on their
> task types though. For example, worker tasks shouldn't run after the global
> scope is closed, while thread control tasks should run even after that. To
> achieve it, worker tasks were always wrapped with WorkerThread's method to
check
> the current thread state. This was so tricky.
> 
> After this CL, each global scope has its own task queues like per-frame task
> scheduler. Worker tasks are pushed into the task queues, control tasks are
still
> pushed into the thread's default task queue though. Thanks to this separation,
> we can abort only worker tasks by simply unregistering the queues when a
global
> scope is closed.
> 
> In addition, this CL unifies postTask interfaces into TaskRunnerHelper (issue
> 694914), and enables any threads to post a task to worker threads because of
> thread-safe refcounted WebTaskRunner.
> 
> BUG=670534, 694914, 710364
> 
> Review-Url: https://codereview.chromium.org/2806623004
> Cr-Commit-Position: refs/heads/master@{#465841}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d56a35fb25dc388a0506174b9e3f...

TBR=kinuko@chromium.org,haraken@chromium.org,tzik@chromium.org,skyostil@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=670534, 694914, 710364
==========

to

==========
Revert of Worker: Introduce per-global-scope task scheduler (patchset #22
id:450001 of https://codereview.chromium.org/2806623004/ )

Reason for revert:
Seems to have caused a crash in fast/workers/termination-early.html on Webkit
Linux Trusty Leak -
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Tru...
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Tru...
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

Original issue's description:
> Worker: Introduce per-global-scope task scheduler
> 
> This CL introduces WorkerGlobalScopeScheduler to make it simpler and saner to
> manage tasks on worker threads.
> 
> Before this CL, tasks on worker threads were always pushed into the thread's
> default task queue, they should be handled in different ways based on their
> task types though. For example, worker tasks shouldn't run after the global
> scope is closed, while thread control tasks should run even after that. To
> achieve it, worker tasks were always wrapped with WorkerThread's method to
check
> the current thread state. This was so tricky.
> 
> After this CL, each global scope has its own task queues like per-frame task
> scheduler. Worker tasks are pushed into the task queues, control tasks are
still
> pushed into the thread's default task queue though. Thanks to this separation,
> we can abort only worker tasks by simply unregistering the queues when a
global
> scope is closed.
> 
> In addition, this CL unifies postTask interfaces into TaskRunnerHelper (issue
> 694914), and enables any threads to post a task to worker threads because of
> thread-safe refcounted WebTaskRunner.
> 
> BUG=670534, 694914, 710364
> 
> Review-Url: https://codereview.chromium.org/2806623004
> Cr-Commit-Position: refs/heads/master@{#465841}
> Committed:
https://chromium.googlesource.com/chromium/src/+/d56a35fb25dc388a0506174b9e3f...

TBR=kinuko@chromium.org,haraken@chromium.org,tzik@chromium.org,skyostil@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=670534, 694914, 710364

Review-Url: https://codereview.chromium.org/2831843002
Cr-Commit-Position: refs/heads/master@{#465892}
Committed:
https://chromium.googlesource.com/chromium/src/+/ee6a4f06b0e0561618669ed56e59...
==========

[commit-bot: I haz the power, 2017-04-20 03:47:03]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ee6a4f06b0e0561618669ed56e59...

[nhiroki, 2017-04-20 04:04:14]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2832763002/ by nhiroki@chromium.org.

The reason for reverting is: Retry.