Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=667310, 671084
Committed: https://crrev.com/200baa21d49f1a2688a33bb64f4b689fdae69119
Cr-Commit-Position: refs/heads/master@{#438146}

[tzik, 2016-12-13 08:54:36]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 08:54:54]

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

[tzik, 2016-12-13 09:09:38]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 09:09:41]

Dry run: 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, 2016-12-13 10:22:59]

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

[commit-bot: I haz the power, 2016-12-13 10:23:00]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-12-13 10:42:58]

Description was changed from

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

BUG=
==========

to

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=
==========

[tzik, 2016-12-13 10:43:13]

tzik@chromium.org changed reviewers:
+ nhiroki@chromium.org

[tzik, 2016-12-13 10:43:14]

PTAL

[nhiroki, 2016-12-13 11:07:16]

Description was changed from

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=
==========

to

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=667310, 671084
==========

[nhiroki, 2016-12-13 11:16:14]

LGTM. Thank you for working on this :)

[tzik, 2016-12-13 11:16:45]

tzik@chromium.org changed reviewers:
+ haraken@chromium.org

[tzik, 2016-12-13 11:17:30]

tzik@chromium.org changed reviewers:
+ yhirano@chromium.org

[tzik, 2016-12-13 11:17:30]

+haraken & yhirano for OWNER review. PTAL.

[haraken, 2016-12-13 11:27:22]

LGTM

[yhirano, 2016-12-13 12:26:44]

lgtm

[tzik, 2016-12-13 12:54:46]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2016-12-13 12:55:01]

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, 2016-12-13 12:59:41]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481633686064870,
"parent_rev": "a60462c5396e8572c43c127f5d6c7796b846e5ff", "commit_rev":
"b90d5cf015e336de520a025c95ec33e8a20b0211"}

[commit-bot: I haz the power, 2016-12-13 13:00:07]

Description was changed from

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=667310, 671084
==========

to

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=667310, 671084

Review-Url: https://codereview.chromium.org/2574703002
==========

[commit-bot: I haz the power, 2016-12-13 13:00:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-13 13:02:46]

Description was changed from

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=667310, 671084

Review-Url: https://codereview.chromium.org/2574703002
==========

to

==========
Use ParentFrameTaskRunners in WorkerLoaderProxyProvider::postTaskToLoader impls

Implementations of WorkerLoaderProxyProvider::postTaskToLoader have used
ExecutionContext::postTask to post tasks to the main thread. However,
the posted tasks are unexpectedly throttled, and that delays the startup
time of service workers.
This CL replaces ExecutionContext::postTask there with ParentFrameTaskRunners
and WebTaskRunner::postTask, so that the posted task is routed to suitable
task runners.

BUG=667310, 671084

Committed: https://crrev.com/200baa21d49f1a2688a33bb64f4b689fdae69119
Cr-Commit-Position: refs/heads/master@{#438146}
==========

[commit-bot: I haz the power, 2016-12-13 13:02:47]

Patchset 2 (id:??) landed as
https://crrev.com/200baa21d49f1a2688a33bb64f4b689fdae69119
Cr-Commit-Position: refs/heads/master@{#438146}