Move unittests in content/common/gpu/client to gpu/ipc/client

Move unittests in content/common/gpu/client to gpu/ipc/client

BUG=586380
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/c9c73f48e2737e2b1038eb14ab0c8f053fb3f251
Cr-Commit-Position: refs/heads/master@{#389823}

[Peng, 2016-04-07 16:49:14]

Description was changed from

==========
WIP


WIP

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

to

==========
WIP


WIP

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-04-12 17:43:09]

Description was changed from

==========
WIP


WIP

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move unittests in content/common/gpu/client to gpu/ipc/client

BUG=586380
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Peng, 2016-04-12 17:45:36]

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

[Peng, 2016-04-12 17:45:36]

hi piman, PTAL. Thanks.

[piman, 2016-04-14 01:31:04]

https://codereview.chromium.org/1867873002/diff/160001/content/browser/androi...
File content/browser/android/browser_surface_texture_manager.h (left):

https://codereview.chromium.org/1867873002/diff/160001/content/browser/androi...
content/browser/android/browser_surface_texture_manager.h:19: public
gpu::SurfaceTexturePeer {
This change and many others in this CL seems unrelated to the topic (moving
unittests). Are they intended?

[Peng, 2016-04-14 15:06:52]

https://codereview.chromium.org/1867873002/diff/160001/content/browser/androi...
File content/browser/android/browser_surface_texture_manager.h (left):

https://codereview.chromium.org/1867873002/diff/160001/content/browser/androi...
content/browser/android/browser_surface_texture_manager.h:19: public
gpu::SurfaceTexturePeer {
On 2016/04/14 01:31:04, piman wrote:
> This change and many others in this CL seems unrelated to the topic (moving
> unittests). Are they intended?

Some unittests on Android platform will crash, if the
BrowserSurfaceTextureManager is not initialized. So to move those unittests to
GPU folder, we have to move BrowserSurfaceTextureManager out of content too.
That why it move this file here.

And this class implements the gpu::SurfaceTexturePeer. The overrided function
EstablishSurfaceTexturePeer() depends on some media player related code in
content. And the media player code has huge dependencies. It is hard to move
them all out of content. I create a new class BrowserSurfaceTexturePeer for that
function and leave it in the content.

[piman, 2016-04-14 22:10:52]

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

[piman, 2016-04-14 22:10:53]

+sievers, could you take look at the android bits? You're more familiar with it
than I am.

[Peng, 2016-04-25 20:46:24]

On 2016/04/14 22:10:53, piman wrote:
> +sievers, could you take look at the android bits? You're more familiar with
it
> than I am.

sievers, kindly ping.

[no sievers, 2016-04-25 23:03:24]

On 2016/04/25 20:46:24, Peng wrote:
> On 2016/04/14 22:10:53, piman wrote:
> > +sievers, could you take look at the android bits? You're more familiar with
> it
> > than I am.
> 
> sievers, kindly ping.

lgtm

[Peng, 2016-04-26 14:12:57]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2016-04-26 14:13:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1867873002/160001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1867873002/160001

[commit-bot: I haz the power, 2016-04-26 14:15:20]

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

[commit-bot: I haz the power, 2016-04-26 14:15:22]

Try jobs failed on following builders:
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Peng, 2016-04-26 15:55:49]

penghuang@chromium.org changed reviewers:
+ jam@chromium.org

[Peng, 2016-04-26 15:55:50]

+jam for .gn, BUILD.gn, testing/* and build/*

hi John, PTAL. Thanks.

[jam, 2016-04-26 16:28:55]

lgtm

[Peng, 2016-04-26 17:45:41]

The CQ bit was checked by penghuang@chromium.org

[Peng, 2016-04-26 17:45:42]

The patchset sent to the CQ was uploaded after l-g-t-m from sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1867873002/#ps200001
(title: "Update")

[commit-bot: I haz the power, 2016-04-26 17:46:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1867873002/200001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1867873002/200001

[commit-bot: I haz the power, 2016-04-26 18:52:30]

Description was changed from

==========
Move unittests in content/common/gpu/client to gpu/ipc/client

BUG=586380
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move unittests in content/common/gpu/client to gpu/ipc/client

BUG=586380
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-04-26 18:52:31]

Committed patchset #11 (id:200001)

[commit-bot: I haz the power, 2016-04-26 18:54:07]

Description was changed from

==========
Move unittests in content/common/gpu/client to gpu/ipc/client

BUG=586380
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move unittests in content/common/gpu/client to gpu/ipc/client

BUG=586380
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/c9c73f48e2737e2b1038eb14ab0c8f053fb3f251
Cr-Commit-Position: refs/heads/master@{#389823}
==========

[commit-bot: I haz the power, 2016-04-26 18:54:08]

Patchset 11 (id:??) landed as
https://crrev.com/c9c73f48e2737e2b1038eb14ab0c8f053fb3f251
Cr-Commit-Position: refs/heads/master@{#389823}