Move more files to gpu/ipc/common

Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.

BUG=596290
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel

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

[Fady Samuel, 2016-03-21 23:10:25]

Description was changed from

==========
[WIP] Move more files to gpu/ipc/common
==========

to

==========
[WIP] Move more files to gpu/ipc/common
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-03-22 02:42:42]

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

[commit-bot: I haz the power, 2016-03-22 02:42:56]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823763003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823763003/1

[commit-bot: I haz the power, 2016-03-22 02:44:47]

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

[commit-bot: I haz the power, 2016-03-22 02:44:48]

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

[Fady Samuel, 2016-03-22 11:53:50]

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

[commit-bot: I haz the power, 2016-03-22 11:53:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823763003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823763003/40001

[Fady Samuel, 2016-03-22 11:57:04]

Description was changed from

==========
[WIP] Move more files to gpu/ipc/common
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_process_launch_causes.h
content/common/gpu/gpu_surface_lookup.ccg
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.
==========

[Fady Samuel, 2016-03-22 11:57:15]

Description was changed from

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_process_launch_causes.h
content/common/gpu/gpu_surface_lookup.ccg
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.
==========

to

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_process_launch_causes.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.
==========

[Fady Samuel, 2016-03-22 11:59:32]

fsamuel@chromium.org changed reviewers:
+ dcheng@chromium.org, piman@chromium.org

[Fady Samuel, 2016-03-22 11:59:33]

+piman@ for move review
+dcheng@ for messages review

PTAL! Thanks! This is a fairly straightforward patch. Hopefully. I'll try to
keep them small to spare myself the accumulated pain of fighting the build
system.

[commit-bot: I haz the power, 2016-03-22 12:10:10]

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

[commit-bot: I haz the power, 2016-03-22 12:10:12]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[Fady Samuel, 2016-03-22 12:32:41]

Description was changed from

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_process_launch_causes.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.
==========

to

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_process_launch_causes.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-03-22 12:32:55]

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

[commit-bot: I haz the power, 2016-03-22 12:33:09]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823763003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823763003/60001

[commit-bot: I haz the power, 2016-03-22 13:32:12]

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

[commit-bot: I haz the power, 2016-03-22 13:32:13]

Dry run: This issue passed the CQ dry run.

[piman, 2016-03-22 15:14:55]

https://codereview.chromium.org/1823763003/diff/60001/gpu/ipc/common/gpu_proc...
File gpu/ipc/common/gpu_process_launch_causes.h (right):

https://codereview.chromium.org/1823763003/diff/60001/gpu/ipc/common/gpu_proc...
gpu/ipc/common/gpu_process_launch_causes.h:8: namespace gpu {
Did this file need to move? It doesn't look like it's used in content/common/gpu
or files that would need to move to gpu/ipc. Since it's only used to establish
the channel (and it kinda hard-codes content-specific concepts), I would prefer
to keep it in content/ if possible.

[piman, 2016-03-22 15:14:55]

https://codereview.chromium.org/1823763003/diff/60001/gpu/ipc/common/gpu_proc...
File gpu/ipc/common/gpu_process_launch_causes.h (right):

https://codereview.chromium.org/1823763003/diff/60001/gpu/ipc/common/gpu_proc...
gpu/ipc/common/gpu_process_launch_causes.h:8: namespace gpu {
Did this file need to move? It doesn't look like it's used in content/common/gpu
or files that would need to move to gpu/ipc. Since it's only used to establish
the channel (and it kinda hard-codes content-specific concepts), I would prefer
to keep it in content/ if possible.

[Fady Samuel, 2016-03-22 21:53:47]

Description was changed from

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_process_launch_causes.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.

BUG=596290
==========

[Fady Samuel, 2016-03-22 21:54:21]

PTAL Antoine! Thanks!

https://codereview.chromium.org/1823763003/diff/60001/gpu/ipc/common/gpu_proc...
File gpu/ipc/common/gpu_process_launch_causes.h (right):

https://codereview.chromium.org/1823763003/diff/60001/gpu/ipc/common/gpu_proc...
gpu/ipc/common/gpu_process_launch_causes.h:8: namespace gpu {
On 2016/03/22 15:14:55, piman wrote:
> Did this file need to move? It doesn't look like it's used in
content/common/gpu
> or files that would need to move to gpu/ipc. Since it's only used to establish
> the channel (and it kinda hard-codes content-specific concepts), I would
prefer
> to keep it in content/ if possible.

I moved this to content/common in another CL. Removed this change in this CL
PTAL.

[piman, 2016-03-22 22:07:06]

lgtm

[Fady Samuel, 2016-03-22 22:08:28]

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

[commit-bot: I haz the power, 2016-03-22 22:08:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823763003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823763003/80001

[dcheng, 2016-03-22 22:11:23]

lgtm

[commit-bot: I haz the power, 2016-03-22 22:21:04]

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

[commit-bot: I haz the power, 2016-03-22 22:21:05]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Fady Samuel, 2016-03-23 01:01:45]

Description was changed from

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.

BUG=596290
==========

to

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.

BUG=596290
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Fady Samuel, 2016-03-23 01:16:50]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-03-23 01:16:51]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org,
piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1823763003/#ps120001
(title: "Cleanup")

[commit-bot: I haz the power, 2016-03-23 01:17:30]

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

[commit-bot: I haz the power, 2016-03-23 01:29:51]

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

[commit-bot: I haz the power, 2016-03-23 01:29:53]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[Fady Samuel, 2016-03-23 02:04:00]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-03-23 02:04:01]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org,
piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1823763003/#ps140001
(title: "Fixed GYP build")

[commit-bot: I haz the power, 2016-03-23 02:04:21]

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

[commit-bot: I haz the power, 2016-03-23 03:41:13]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-03-23 03:42:29]

Description was changed from

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.

BUG=596290
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move more files to gpu/ipc/common

This CL moves:

content/common/gpu/gpu_memory_uma_stats.h
content/common/gpu/gpu_surface_lookup.cc
content/common/gpu/gpu_surface_lookup.h

to gpu/ipc/common.

BUG=596290
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel

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

[commit-bot: I haz the power, 2016-03-23 03:42:31]

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