Reduce createSameThreadTask usage in modules/

Reduce createSameThreadTask usage in modules/

ExecutionContextTask is deprecated and will be replaced with
WTF::Closure. This patch replaces ExecutionContextTask::postTask
with WebTaskRunner::postTask in some of its occurrences.

BUG=625927
Review-Url: https://codereview.chromium.org/2698153003
Cr-Commit-Position: refs/heads/master@{#451268}
Committed: https://chromium.googlesource.com/chromium/src/+/9d20a4962d5ffabb59953294bb28c6c9799422a8

[yuryu, 2017-02-16 09:09:27]

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

[commit-bot: I haz the power, 2017-02-16 09:10:09]

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, 2017-02-16 13:02:00]

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

[commit-bot: I haz the power, 2017-02-16 13:02:01]

Dry run: This issue passed the CQ dry run.

[yuryu, 2017-02-16 15:02:10]

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

[yuryu, 2017-02-16 15:02:11]

Hi, can you take a look?

[haraken, 2017-02-16 15:31:46]

LGTM

[nhiroki, 2017-02-17 03:47:15]

lgtm

https://codereview.chromium.org/2698153003/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp
(right):

https://codereview.chromium.org/2698153003/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp:64: static
void runNotifyInitialAvailabilityTask(
nit: Can you move this into the anonymous namespace above instead of static
function? Using static function is also allowed[1], but it'd better to make it
consistent with existing code.

[1]
https://google.github.io/styleguide/cppguide.html#Unnamed_Namespaces_and_Stat...

[tzik, 2017-02-17 04:34:19]

lgtm

[yuryu, 2017-02-17 05:14:33]

https://codereview.chromium.org/2698153003/diff/1/third_party/WebKit/Source/m...
File third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp
(right):

https://codereview.chromium.org/2698153003/diff/1/third_party/WebKit/Source/m...
third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.cpp:64: static
void runNotifyInitialAvailabilityTask(
On 2017/02/17 03:47:15, nhiroki wrote:
> nit: Can you move this into the anonymous namespace above instead of static
> function? Using static function is also allowed[1], but it'd better to make it
> consistent with existing code.
> 
> [1]
>
https://google.github.io/styleguide/cppguide.html#Unnamed_Namespaces_and_Stat...

Done.

[yuryu, 2017-02-17 05:15:31]

The CQ bit was checked by yuryu@chromium.org

[yuryu, 2017-02-17 05:15:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, nhiroki@chromium.org, tzik@chromium.org
Link to the patchset: https://codereview.chromium.org/2698153003/#ps20001
(title: "Move a static callback to an anonymous namespace")

[commit-bot: I haz the power, 2017-02-17 05:16:04]

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-02-17 07:13:32]

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

[commit-bot: I haz the power, 2017-02-17 07:13:33]

Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yuryu, 2017-02-17 07:32:17]

The CQ bit was checked by yuryu@chromium.org

[commit-bot: I haz the power, 2017-02-17 07:32:44]

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-02-17 08:27:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487316737803290,
"parent_rev": "5f6e56e59eb16be722d9424dafa03d19a2aa6697", "commit_rev":
"9d20a4962d5ffabb59953294bb28c6c9799422a8"}

[commit-bot: I haz the power, 2017-02-17 08:27:44]

Description was changed from

==========
Reduce createSameThreadTask usage in modules/

ExecutionContextTask is deprecated and will be replaced with
WTF::Closure. This patch replaces ExecutionContextTask::postTask
with WebTaskRunner::postTask in some of its occurrences.

BUG=625927
==========

to

==========
Reduce createSameThreadTask usage in modules/

ExecutionContextTask is deprecated and will be replaced with
WTF::Closure. This patch replaces ExecutionContextTask::postTask
with WebTaskRunner::postTask in some of its occurrences.

BUG=625927

Review-Url: https://codereview.chromium.org/2698153003
Cr-Commit-Position: refs/heads/master@{#451268}
Committed:
https://chromium.googlesource.com/chromium/src/+/9d20a4962d5ffabb59953294bb28...
==========

[commit-bot: I haz the power, 2017-02-17 08:27:46]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/9d20a4962d5ffabb59953294bb28...