gpu: Add a couple of unit-tests for GpuService.

gpu: Add a couple of unit-tests for GpuService.

Add a couple of unit-tests to verify that GpuService can be destroyed
safely both after incoming GpuServiceRequest have been bound, or while
they are scheduled to be bound.

BUG=706375, 706357
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2787363003
Cr-Commit-Position: refs/heads/master@{#461658}
Committed: https://chromium.googlesource.com/chromium/src/+/94d7bbdbc5110011a16039c47afa60c4ee065687

[sadrul, 2017-04-01 03:27:25]

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

[commit-bot: I haz the power, 2017-04-01 03:27:37]

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-04-01 03:31:09]

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

[commit-bot: I haz the power, 2017-04-01 03:31:10]

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

[sadrul, 2017-04-01 03:34:47]

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

[commit-bot: I haz the power, 2017-04-01 03:34:57]

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-04-01 04:11:07]

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

[commit-bot: I haz the power, 2017-04-01 04:11:08]

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

[sadrul, 2017-04-01 06:07:03]

Description was changed from

==========
gpu: Add a couple of unit-tests for GpuService.

Add a couple of unit-tests to verify that GpuService can be destroyed
safely both after incoming GpuServiceRequest have been bound, or while
they are scheduled to be bound.

BUG=706375, 706357
==========

to

==========
gpu: Add a couple of unit-tests for GpuService.

Add a couple of unit-tests to verify that GpuService can be destroyed
safely both after incoming GpuServiceRequest have been bound, or while
they are scheduled to be bound.

BUG=706375, 706357
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[sadrul, 2017-04-01 06:07:05]

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

[commit-bot: I haz the power, 2017-04-01 06:07: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-04-01 08:09:39]

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

[commit-bot: I haz the power, 2017-04-01 08:09:40]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[sadrul, 2017-04-03 15:25:26]

sadrul@chromium.org changed reviewers:
+ piman@chromium.org

[piman, 2017-04-03 23:55:46]

lgtm

[sadrul, 2017-04-04 02:05:38]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2017-04-04 02:06:14]

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-04-04 02:38:46]

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

[commit-bot: I haz the power, 2017-04-04 02:38:47]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sadrul, 2017-04-04 06:04:44]

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

[commit-bot: I haz the power, 2017-04-04 06:05:01]

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-04-04 07:50:09]

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

[commit-bot: I haz the power, 2017-04-04 07:50:10]

Dry run: This issue passed the CQ dry run.

[sadrul, 2017-04-04 08:06:20]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2017-04-04 08:06:21]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/2787363003/#ps60001
(title: "tot merge")

[commit-bot: I haz the power, 2017-04-04 08:06:35]

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-04-04 08:11:20]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1491293180674210,
"parent_rev": "519ead74e98302eabc1144ed9947e14d93adae95", "commit_rev":
"94d7bbdbc5110011a16039c47afa60c4ee065687"}

[commit-bot: I haz the power, 2017-04-04 08:12:06]

Description was changed from

==========
gpu: Add a couple of unit-tests for GpuService.

Add a couple of unit-tests to verify that GpuService can be destroyed
safely both after incoming GpuServiceRequest have been bound, or while
they are scheduled to be bound.

BUG=706375, 706357
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
gpu: Add a couple of unit-tests for GpuService.

Add a couple of unit-tests to verify that GpuService can be destroyed
safely both after incoming GpuServiceRequest have been bound, or while
they are scheduled to be bound.

BUG=706375, 706357
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2787363003
Cr-Commit-Position: refs/heads/master@{#461658}
Committed:
https://chromium.googlesource.com/chromium/src/+/94d7bbdbc5110011a16039c47afa...
==========

[commit-bot: I haz the power, 2017-04-04 08:12:07]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/94d7bbdbc5110011a16039c47afa...