Remove base::ScopedPtrHashMap from gpu/ipc/service/.

Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
CQ_INCLUDE_TRYBOTS=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

Committed: https://crrev.com/4e9384e00eeb05c2558cf731b7494b14d3d5e8bf
Cr-Commit-Position: refs/heads/master@{#440941}

[Avi (use Gerrit), 2016-12-28 21:52:34]

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

[Avi (use Gerrit), 2016-12-28 21:52:35]

Description was changed from

==========
Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
==========

to

==========
Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
CQ_INCLUDE_TRYBOTS=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
==========

[commit-bot: I haz the power, 2016-12-28 21:52:48]

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, 2016-12-28 22:39:27]

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

[commit-bot: I haz the power, 2016-12-28 22:39:28]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2016-12-28 22:49:37]

avi@chromium.org changed reviewers:
+ jbauman@chromium.org, kbr@chromium.org, vmiura@chromium.org, zmo@chromium.org

[Avi (use Gerrit), 2016-12-28 22:49:38]

To any reviewer in the office.

[Ken Russell (switch to Gerrit), 2016-12-29 05:41:15]

lgtm

https://codereview.chromium.org/2601993002/diff/1/gpu/ipc/service/gpu_channel.cc
File gpu/ipc/service/gpu_channel.cc (right):

https://codereview.chromium.org/2601993002/diff/1/gpu/ipc/service/gpu_channel...
gpu/ipc/service/gpu_channel.cc:903: stubs_[route_id] = std::move(stub);
I confirmed that forgetting the std::move here is a compiler error, so this
looks fine.

[Avi (use Gerrit), 2016-12-29 06:00:20]

The CQ bit was checked by avi@chromium.org

[commit-bot: I haz the power, 2016-12-29 06:00:33]

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-12-29 06:04:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482991220738150, "parent_rev":
"517ce71a2bfa5e23da1bc75315052ba4688c4733", "commit_rev":
"bdcae291a5c81774063b8d120c38282d181120e7"}

[commit-bot: I haz the power, 2016-12-29 06:05:22]

Description was changed from

==========
Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
CQ_INCLUDE_TRYBOTS=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

==========
Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
CQ_INCLUDE_TRYBOTS=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/2601993002
==========

[commit-bot: I haz the power, 2016-12-29 06:05:23]

Committed patchset #1 (id:1)

[Avi (use Gerrit), 2016-12-29 15:18:02]

https://codereview.chromium.org/2601993002/diff/1/gpu/ipc/service/gpu_channel.cc
File gpu/ipc/service/gpu_channel.cc (right):

https://codereview.chromium.org/2601993002/diff/1/gpu/ipc/service/gpu_channel...
gpu/ipc/service/gpu_channel.cc:903: stubs_[route_id] = std::move(stub);
On 2016/12/29 05:41:15, Ken Russell OOO-till-Jan-3 wrote:
> I confirmed that forgetting the std::move here is a compiler error, so this
> looks fine.

Acknowledged.

[commit-bot: I haz the power, 2017-01-02 15:51:52]

Description was changed from

==========
Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
CQ_INCLUDE_TRYBOTS=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/2601993002
==========

to

==========
Remove base::ScopedPtrHashMap from gpu/ipc/service/.

BUG=579229
CQ_INCLUDE_TRYBOTS=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

Committed: https://crrev.com/4e9384e00eeb05c2558cf731b7494b14d3d5e8bf
Cr-Commit-Position: refs/heads/master@{#440941}
==========

[commit-bot: I haz the power, 2017-01-02 15:51:53]

Patchset 1 (id:??) landed as
https://crrev.com/4e9384e00eeb05c2558cf731b7494b14d3d5e8bf
Cr-Commit-Position: refs/heads/master@{#440941}