Remove WebCallbackTask

Remove WebCallbackTask

Since WebTaskRunner can handle base::Closure, WebCallbackTask is no
longer needed.

Committed: https://crrev.com/5f3ffb296e91d9616af89883b9d64bfa1be60b34
Cr-Commit-Position: refs/heads/master@{#420083}

[tzik, 2016-09-21 09:21:43]

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

[commit-bot: I haz the power, 2016-09-21 09:21: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-09-21 09:24:35]

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

[commit-bot: I haz the power, 2016-09-21 09:24:44]

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

[tzik, 2016-09-21 09:29:22]

Description was changed from

==========
Remove WebCallbackTask

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

to

==========
Remove WebCallbackTask

Since WebTaskRunner can handle base::Closure, WebCallbackTask is no
longer needed.
==========

[tzik, 2016-09-21 09:29:22]

tzik@chromium.org changed reviewers:
+ alexclarke@chromium.org, mkwst@chromium.org

[tzik, 2016-09-21 10:13:09]

PTAL

[jochen (gone - plz use gerrit), 2016-09-21 10:18:57]

jochen@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-09-21 10:18:57]

lgtm

[alex clarke (OOO till 29th), 2016-09-21 10:38:40]

https://codereview.chromium.org/2357043003/diff/20001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebTaskRunner.h (right):

https://codereview.chromium.org/2357043003/diff/20001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebTaskRunner.h:45: virtual void
postTask(const WebTraceLocation&, const base::Closure&) = 0;
Getting rid of WebCallbackTask is a good idea, but do we really need this?  We
already have toSingleThreadTaskRunner().

[commit-bot: I haz the power, 2016-09-21 11:20:41]

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

[commit-bot: I haz the power, 2016-09-21 11:20:42]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-09-21 14:29:34]

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

[commit-bot: I haz the power, 2016-09-21 14:29:52]

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

[tzik, 2016-09-21 14:32:09]

https://codereview.chromium.org/2357043003/diff/20001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/WebTaskRunner.h (right):

https://codereview.chromium.org/2357043003/diff/20001/third_party/WebKit/publ...
third_party/WebKit/public/platform/WebTaskRunner.h:45: virtual void
postTask(const WebTraceLocation&, const base::Closure&) = 0;
On 2016/09/21 10:38:40, alex clarke wrote:
> Getting rid of WebCallbackTask is a good idea, but do we really need this?  We
> already have toSingleThreadTaskRunner().

Ah, right. Updated. We can use SingleThreadTaskRunner there.

[alex clarke (OOO till 29th), 2016-09-21 15:42:17]

lgtm

[commit-bot: I haz the power, 2016-09-21 15:53:32]

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

[commit-bot: I haz the power, 2016-09-21 15:53:33]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-09-21 16:48:09]

The CQ bit was checked by tzik@chromium.org

[tzik, 2016-09-21 16:48:09]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2357043003/#ps40001
(title: "remove WebTaskRunner::postTask(base::Closure). git cl format")

[commit-bot: I haz the power, 2016-09-21 16:48: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, 2016-09-21 16:55:11]

Description was changed from

==========
Remove WebCallbackTask

Since WebTaskRunner can handle base::Closure, WebCallbackTask is no
longer needed.
==========

to

==========
Remove WebCallbackTask

Since WebTaskRunner can handle base::Closure, WebCallbackTask is no
longer needed.
==========

[commit-bot: I haz the power, 2016-09-21 16:55:12]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-21 16:56:37]

Description was changed from

==========
Remove WebCallbackTask

Since WebTaskRunner can handle base::Closure, WebCallbackTask is no
longer needed.
==========

to

==========
Remove WebCallbackTask

Since WebTaskRunner can handle base::Closure, WebCallbackTask is no
longer needed.

Committed: https://crrev.com/5f3ffb296e91d9616af89883b9d64bfa1be60b34
Cr-Commit-Position: refs/heads/master@{#420083}
==========

[commit-bot: I haz the power, 2016-09-21 16:56:38]

Patchset 3 (id:??) landed as
https://crrev.com/5f3ffb296e91d9616af89883b9d64bfa1be60b34
Cr-Commit-Position: refs/heads/master@{#420083}