Don't move raw pointers in gpu

Don't move raw pointers in gpu

Moving raw pointers does the same job as copying, but is more verbose and also
more confusing: e.g., is the supposed pointer meant to be a smart one?

This instance was flagged by the tool from
https://codereview.chromium.org/2919243002/.

BUG=729393
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/2920133002
Cr-Commit-Position: refs/heads/master@{#477233}
Committed: https://chromium.googlesource.com/chromium/src/+/e38dbf83761a9f1ee952e4f908040d50b34f0b12

[vabr (Chromium), 2017-06-04 11:15:30]

Description was changed from

==========
Don't move raw pointers in gpu

Moving raw pointers does the same job as copying, but is more verbose and also
more confusing: e.g., is the supposed pointer meant to be a smart one?

This instance was flagged by the tool from
https://codereview.chromium.org/2919243002/.

BUG=729393
==========

to

==========
Don't move raw pointers in gpu

Moving raw pointers does the same job as copying, but is more verbose and also
more confusing: e.g., is the supposed pointer meant to be a smart one?

This instance was flagged by the tool from
https://codereview.chromium.org/2919243002/.

BUG=729393
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
==========

[vabr (Chromium), 2017-06-04 11:15:52]

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

[commit-bot: I haz the power, 2017-06-04 11:16:00]

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-06-04 12:15:30]

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

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

Dry run: This issue passed the CQ dry run.

[vabr (Chromium), 2017-06-04 12:25:55]

vabr@chromium.org changed reviewers:
+ jbauman@chromium.org

[vabr (Chromium), 2017-06-04 12:25:56]

Hi jbauman@,
Could you please review?
Thanks,
Vaclav

[jbauman, 2017-06-05 20:17:12]

lgtm

[vabr (Chromium), 2017-06-06 06:44:31]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2017-06-06 06:44:58]

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-06-06 08:23:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496731471097680, "parent_rev":
"5f5c0dba3c72ae92899db2b355ca36c9da7b49fb", "commit_rev":
"e38dbf83761a9f1ee952e4f908040d50b34f0b12"}

[commit-bot: I haz the power, 2017-06-06 08:23:49]

Description was changed from

==========
Don't move raw pointers in gpu

Moving raw pointers does the same job as copying, but is more verbose and also
more confusing: e.g., is the supposed pointer meant to be a smart one?

This instance was flagged by the tool from
https://codereview.chromium.org/2919243002/.

BUG=729393
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

==========
Don't move raw pointers in gpu

Moving raw pointers does the same job as copying, but is more verbose and also
more confusing: e.g., is the supposed pointer meant to be a smart one?

This instance was flagged by the tool from
https://codereview.chromium.org/2919243002/.

BUG=729393
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/2920133002
Cr-Commit-Position: refs/heads/master@{#477233}
Committed:
https://chromium.googlesource.com/chromium/src/+/e38dbf83761a9f1ee952e4f90804...
==========

[commit-bot: I haz the power, 2017-06-06 08:23:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e38dbf83761a9f1ee952e4f90804...