Fix for a DCHECK in GpuSurfaceLookup

GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it would end up with
multiple instances of the GpuSurfaceTracker singleton.
GpuSurfaceTracker inherits from GpuSurfaceLookup for which there is only
one instance (part of the gpu component), causing a DCHECK.
This CL moves the GpuSurfaceTracker to the gpu component like
GpuSurfaceLookup.

BUG=661333
TEST=content_shell should still work on Android and Mac.

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

[Jay Civelli, 2016-11-01 21:37:09]

Description was changed from

==========
Make SurfaceTracker part of the ipc/common code.

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

to

==========
Make SurfaceTracker part of the ipc/common code.

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

[Jay Civelli, 2016-11-01 21:44:51]

Description was changed from

==========
Make SurfaceTracker part of the ipc/common code.

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

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it end up with
multiple instances of the singleton which extends GpuSurfaceLookup for
which there is only one instance (part of the gpu component), causing
a DCHECK.

BUG=661333
TEST=content_shell should still work on Android and Mac.
==========

[Jay Civelli, 2016-11-01 21:49:18]

Description was changed from

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it end up with
multiple instances of the singleton which extends GpuSurfaceLookup for
which there is only one instance (part of the gpu component), causing
a DCHECK.

BUG=661333
TEST=content_shell should still work on Android and Mac.
==========

to

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it would end up with
multiple instances of the GpuSurfaceTracker singleton.
GpuSurfaceTracker inherits from GpuSurfaceLookup for which there is only
one instance (part of the gpu component), causing a DCHECK.
This CL moves the GpuSurfaceTracker to the gpu component like
GpuSurfaceLookup.

BUG=661333
TEST=content_shell should still work on Android and Mac.
==========

[Jay Civelli, 2016-11-01 22:03:30]

jcivelli@chromium.org changed reviewers:
+ boliu@chromium.org

[boliu, 2016-11-01 22:09:31]

lgtm

[Jay Civelli, 2016-11-01 22:54:10]

jcivelli@chromium.org changed reviewers:
+ ochang@chromium.org

[Jay Civelli, 2016-11-01 22:55:10]

Adding @ochang for OWNER's review of gpu/ipc/common

Thanks.

[Oliver Chang, 2016-11-01 23:16:21]

lgtm

[Jay Civelli, 2016-11-01 23:17:27]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2016-11-01 23:18:10]

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-02 01:02:31]

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

[commit-bot: I haz the power, 2016-11-02 01:02:32]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Jay Civelli, 2016-11-02 04:12:42]

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

[commit-bot: I haz the power, 2016-11-02 04:13: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, 2016-11-02 04:45:27]

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

[commit-bot: I haz the power, 2016-11-02 04:45:28]

Dry run: This issue passed the CQ dry run.

[Jay Civelli, 2016-11-02 18:38:34]

The CQ bit was checked by jcivelli@chromium.org

[commit-bot: I haz the power, 2016-11-02 18:39:03]

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-02 18:44:33]

Description was changed from

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it would end up with
multiple instances of the GpuSurfaceTracker singleton.
GpuSurfaceTracker inherits from GpuSurfaceLookup for which there is only
one instance (part of the gpu component), causing a DCHECK.
This CL moves the GpuSurfaceTracker to the gpu component like
GpuSurfaceLookup.

BUG=661333
TEST=content_shell should still work on Android and Mac.
==========

to

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it would end up with
multiple instances of the GpuSurfaceTracker singleton.
GpuSurfaceTracker inherits from GpuSurfaceLookup for which there is only
one instance (part of the gpu component), causing a DCHECK.
This CL moves the GpuSurfaceTracker to the gpu component like
GpuSurfaceLookup.

BUG=661333
TEST=content_shell should still work on Android and Mac.
==========

[commit-bot: I haz the power, 2016-11-02 18:44:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-02 18:54:05]

Description was changed from

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it would end up with
multiple instances of the GpuSurfaceTracker singleton.
GpuSurfaceTracker inherits from GpuSurfaceLookup for which there is only
one instance (part of the gpu component), causing a DCHECK.
This CL moves the GpuSurfaceTracker to the gpu component like
GpuSurfaceLookup.

BUG=661333
TEST=content_shell should still work on Android and Mac.
==========

to

==========
GpuSurfaceTracker should not be a source set other components depend on.
As it is today, different components depending on it would end up with
multiple instances of the GpuSurfaceTracker singleton.
GpuSurfaceTracker inherits from GpuSurfaceLookup for which there is only
one instance (part of the gpu component), causing a DCHECK.
This CL moves the GpuSurfaceTracker to the gpu component like
GpuSurfaceLookup.

BUG=661333
TEST=content_shell should still work on Android and Mac.

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

[commit-bot: I haz the power, 2016-11-02 18:54:06]

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