Clean up WebTaskRunner::Task subclasses in web/tests/

Clean up WebTaskRunner::Task subclasses in web/tests/

BUG=478194
Committed: https://crrev.com/2882d59b5df13afe6e081828a32ce8ee54731b13
Cr-Commit-Position: refs/heads/master@{#376730}

[hiroshige, 2016-02-20 01:44:23]

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

[commit-bot: I haz the power, 2016-02-20 01:49:56]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/1

[commit-bot: I haz the power, 2016-02-20 02:08:11]

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

[commit-bot: I haz the power, 2016-02-20 02:08:12]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[hiroshige, 2016-02-20 02:14:28]

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

[commit-bot: I haz the power, 2016-02-20 02:16:44]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/20001

[hiroshige, 2016-02-20 02:20:21]

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

[commit-bot: I haz the power, 2016-02-20 02:22:29]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/40001

[commit-bot: I haz the power, 2016-02-20 03:35:50]

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

[commit-bot: I haz the power, 2016-02-20 03:35:51]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-02-21 16:50:31]

Patchset #4 (id:60001) has been deleted

[hiroshige, 2016-02-21 17:00:48]

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

[commit-bot: I haz the power, 2016-02-21 17:01:28]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/80001

[commit-bot: I haz the power, 2016-02-21 17:03:24]

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

[commit-bot: I haz the power, 2016-02-21 17:03:24]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[hiroshige, 2016-02-21 18:08:31]

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

[commit-bot: I haz the power, 2016-02-21 18:08:41]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/100001

[commit-bot: I haz the power, 2016-02-21 18:48:20]

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

[commit-bot: I haz the power, 2016-02-21 18:48:20]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-02-21 18:57:28]

hiroshige@chromium.org changed reviewers:
+ kinuko@chromium.org

[hiroshige, 2016-02-21 18:57:29]

PTAL.

[kinuko, 2016-02-22 04:31:51]

+dcheng for owner review

The postTask's in Web{View,Frame}Test are now gone, lgtm for FrameTestHelper.cpp
change

[hiroshige, 2016-02-22 04:45:10]

hiroshige@chromium.org changed reviewers:
+ dcheng@chromium.org

[hiroshige, 2016-02-22 04:45:11]

dcheng@, PTAL.

[dcheng, 2016-02-22 05:02:28]

https://codereview.chromium.org/1710403003/diff/100001/third_party/WebKit/Sou...
File third_party/WebKit/Source/web/tests/WebFrameTest.cpp (left):

https://codereview.chromium.org/1710403003/diff/100001/third_party/WebKit/Sou...
third_party/WebKit/Source/web/tests/WebFrameTest.cpp:3237: class
ReloadWithOverrideURLTask : public WebTaskRunner::Task {
I think kinuko@ just removed this so you might have to rebase.

https://codereview.chromium.org/1710403003/diff/100001/third_party/WebKit/Sou...
File third_party/WebKit/Source/web/tests/WebViewTest.cpp (left):

https://codereview.chromium.org/1710403003/diff/100001/third_party/WebKit/Sou...
third_party/WebKit/Source/web/tests/WebViewTest.cpp:1348: class DropTask :
public WebTaskRunner::Task {
This one as well.

[hiroshige, 2016-02-22 06:12:27]

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

[hiroshige, 2016-02-22 06:12:47]

Oh, I see. Rebased.

[commit-bot: I haz the power, 2016-02-22 06:13:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/120001

[dcheng, 2016-02-22 07:01:37]

lgtm

[commit-bot: I haz the power, 2016-02-22 07:15:28]

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

[commit-bot: I haz the power, 2016-02-22 07:15:28]

Dry run: This issue passed the CQ dry run.

[hiroshige, 2016-02-22 15:17:59]

The CQ bit was checked by hiroshige@chromium.org

[hiroshige, 2016-02-22 15:18:00]

The patchset sent to the CQ was uploaded after l-g-t-m from kinuko@chromium.org
Link to the patchset: https://codereview.chromium.org/1710403003/#ps120001
(title: "Rebase")

[commit-bot: I haz the power, 2016-02-22 15:18:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1710403003/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1710403003/120001

[commit-bot: I haz the power, 2016-02-22 15:25:59]

Committed patchset #6 (id:120001)

[commit-bot: I haz the power, 2016-02-22 15:26:54]

Description was changed from

==========
Clean up WebTaskRunner::Task subclasses in web/tests/

BUG=478194
==========

to

==========
Clean up WebTaskRunner::Task subclasses in web/tests/

BUG=478194

Committed: https://crrev.com/2882d59b5df13afe6e081828a32ce8ee54731b13
Cr-Commit-Position: refs/heads/master@{#376730}
==========

[commit-bot: I haz the power, 2016-02-22 15:26:55]

Patchset 6 (id:??) landed as
https://crrev.com/2882d59b5df13afe6e081828a32ce8ee54731b13
Cr-Commit-Position: refs/heads/master@{#376730}