gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for trivial change in //blimp

Committed: https://crrev.com/849cb9e4fc81fe1d7249e474f4f9f61b568da052
Cr-Commit-Position: refs/heads/master@{#430958}

[sadrul, 2016-11-09 03:22:24]

Description was changed from

==========
exp: direct cast.

BUG=
==========

to

==========
exp: direct cast.

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

[sadrul, 2016-11-09 03:22:27]

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

[commit-bot: I haz the power, 2016-11-09 03:22:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2016-11-09 04:11:57]

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

[commit-bot: I haz the power, 2016-11-09 04:12:24]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2016-11-09 04:23:50]

Description was changed from

==========
exp: direct cast.

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

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to GpuMemoryBufferImpl instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

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

[sadrul, 2016-11-09 04:26:36]

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

[sadrul, 2016-11-09 04:26:36]

I haven't looked back in history to see if some implementations of
GpuMemoryBufferFromClientBuffer() used to do anything different, but the bots
seem fairly satisfied with a direct cast, and looks like the conversion function
can be safely removed?

[sadrul, 2016-11-09 04:43:00]

Description was changed from

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to GpuMemoryBufferImpl instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

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

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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-11-09 05:04:43]

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

[commit-bot: I haz the power, 2016-11-09 05:04:44]

Dry run: This issue passed the CQ dry run.

[reveman, 2016-11-09 07:53:24]

lgtm

[sadrul, 2016-11-09 12:39:21]

sadrul@chromium.org changed reviewers:
+ khushalsagar@chromium.org, piman@chromium.org, sky@chromium.org

[sadrul, 2016-11-09 12:39:21]

+khushalsagar@ for blimp/ owner

+piman@ for content/ and gpu/ owner

+sky@ for services/ui

[sky, 2016-11-09 16:26:01]

LGTM

[piman, 2016-11-09 16:37:51]

lgtm

[sadrul, 2016-11-09 17:14:19]

Description was changed from

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

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

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for //blimp, since trivual change.
==========

[sadrul, 2016-11-09 17:14:41]

Description was changed from

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for //blimp, since trivual change.
==========

to

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for trivial change in //blimp
==========

[sadrul, 2016-11-09 17:15:09]

The CQ bit was checked by sadrul@chromium.org

[commit-bot: I haz the power, 2016-11-09 17:15: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-11-09 17:20:09]

Description was changed from

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for trivial change in //blimp
==========

to

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for trivial change in //blimp
==========

[commit-bot: I haz the power, 2016-11-09 17:20:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-09 17:26:34]

Description was changed from

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for trivial change in //blimp
==========

to

==========
gpu: Directly cast ClientBuffer to gfx::GpuMemoryBuffer

Directly cast ClientBuffer to gfx::GpuMemoryBuffer instead of going
through GpuMemoryBufferManager::GpuMemoryBufferFromClientBuffer().

BUG=none
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
TBR=khushalsagar@chromium.org for trivial change in //blimp

Committed: https://crrev.com/849cb9e4fc81fe1d7249e474f4f9f61b568da052
Cr-Commit-Position: refs/heads/master@{#430958}
==========

[commit-bot: I haz the power, 2016-11-09 17:26:35]

Patchset 2 (id:??) landed as
https://crrev.com/849cb9e4fc81fe1d7249e474f4f9f61b568da052
Cr-Commit-Position: refs/heads/master@{#430958}