|
|
DescriptionUse base::test::ScopedAsyncTaskScheduler in GLImageTest.
GLImageTest registers a TaskScheduler by calling
base::test::CreateAndSetSimpleTaskScheduler, but never unregisters
it. The second GLImageTest running in fails because there is
already a registered TaskScheduler.
This CL replaces the call to base::test::CreateAndSetSimpleTaskScheduler
with a base::test::ScopedAsyncTaskScheduler member which takes care of
registering a TaskScheduler at the beginning of the test and
unregistering it at the end of the test.
BUG=693203
Review-Url: https://codereview.chromium.org/2708923003
Cr-Commit-Position: refs/heads/master@{#451842}
Committed: https://chromium.googlesource.com/chromium/src/+/8a9cc7e7dc99a62392c3a8fd02ac4c3a56415240
Patch Set 1 #Patch Set 2 : fix build error #Patch Set 3 : add dependency #Patch Set 4 : fix dependency #Patch Set 5 : fix dependency #Messages
Total messages: 35 (28 generated)
The CQ bit was checked by fdoray@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by fdoray@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...) android_cronet on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...) linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...) mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
The CQ bit was checked by fdoray@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...) android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...) linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...) mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...) mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
The CQ bit was checked by fdoray@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...) mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...) mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
The CQ bit was checked by fdoray@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
fdoray@chromium.org changed reviewers: + reveman@chromium.org
PTAL
dcastagna@chromium.org changed reviewers: + dcastagna@chromium.org
LGTM, thanks!
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
kbr@chromium.org changed reviewers: + kbr@chromium.org
Thanks for the fix! LGTM too.
The CQ bit was checked by fdoray@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
The CQ bit was checked by kbr@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 80001, "attempt_start_ts": 1487711684618450, "parent_rev": "310a283e2982013c42652daa4af5c2ad621a5013", "commit_rev": "8a9cc7e7dc99a62392c3a8fd02ac4c3a56415240"}
Message was sent while issue was closed.
Description was changed from ========== Use base::test::ScopedAsyncTaskScheduler in GLImageTest. GLImageTest registers a TaskScheduler by calling base::test::CreateAndSetSimpleTaskScheduler, but never unregisters it. The second GLImageTest running in fails because there is already a registered TaskScheduler. This CL replaces the call to base::test::CreateAndSetSimpleTaskScheduler with a base::test::ScopedAsyncTaskScheduler member which takes care of registering a TaskScheduler at the beginning of the test and unregistering it at the end of the test. BUG=693203 ========== to ========== Use base::test::ScopedAsyncTaskScheduler in GLImageTest. GLImageTest registers a TaskScheduler by calling base::test::CreateAndSetSimpleTaskScheduler, but never unregisters it. The second GLImageTest running in fails because there is already a registered TaskScheduler. This CL replaces the call to base::test::CreateAndSetSimpleTaskScheduler with a base::test::ScopedAsyncTaskScheduler member which takes care of registering a TaskScheduler at the beginning of the test and unregistering it at the end of the test. BUG=693203 Review-Url: https://codereview.chromium.org/2708923003 Cr-Commit-Position: refs/heads/master@{#451842} Committed: https://chromium.googlesource.com/chromium/src/+/8a9cc7e7dc99a62392c3a8fd02ac... ==========
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as https://chromium.googlesource.com/chromium/src/+/8a9cc7e7dc99a62392c3a8fd02ac... |