Mus+Ash: ServerWindowCompositorFrameSink => GpuCompositorFrameSink

Mus+Ash: ServerWindowCompositorFrameSink => GpuCompositorFrameSink

The thing that is currently called ServerWindowCompositorFrameSink doesn't know
and shouldn't know anything about ServerWindows because it'll live in the GPU
process. This CL renames the class to better capture this requirement.

BUG=661278
Committed: https://crrev.com/16700579f33d11511d05966bc1c42f0b90bacd8d
Cr-Commit-Position: refs/heads/master@{#431514}

[Fady Samuel, 2016-11-10 22:18:30]

fsamuel@chromium.org changed reviewers:
+ sadrul@chromium.org, sky@chromium.org

[Fady Samuel, 2016-11-10 22:18:31]

+sky@ for OWNERS
+sadrul@ FYI

[sky, 2016-11-11 00:22:01]

LGTM

[Fady Samuel, 2016-11-11 00:52:57]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-11-11 00:53:34]

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-11 01:22:04]

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

[commit-bot: I haz the power, 2016-11-11 01:22:05]

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

[sadrul, 2016-11-11 01:46:42]

lgtm (X.h has a #define for DestroyAll(). That's what's causing the build
issues)

[Fady Samuel, 2016-11-11 05:00:44]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-11-11 05:00:45]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org,
sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2494663004/#ps60001
(title: "ServerWindowCompositorFrameSinkManager should not reference
SurfaceFactory or SurfaceManager")

[commit-bot: I haz the power, 2016-11-11 05:00:56]

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-11 07:08:53]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-11 07:13:14]

Description was changed from

==========
Mus+Ash: ServerWindowCompositorFrameSink => GpuCompositorFrameSink

The thing that is currently called ServerWindowCompositorFrameSink doesn't know
and shouldn't know anything about ServerWindows because it'll live in the GPU
process. This CL renames the class to better capture this requirement.

BUG=661278
==========

to

==========
Mus+Ash: ServerWindowCompositorFrameSink => GpuCompositorFrameSink

The thing that is currently called ServerWindowCompositorFrameSink doesn't know
and shouldn't know anything about ServerWindows because it'll live in the GPU
process. This CL renames the class to better capture this requirement.

BUG=661278

Committed: https://crrev.com/16700579f33d11511d05966bc1c42f0b90bacd8d
Cr-Commit-Position: refs/heads/master@{#431514}
==========

[commit-bot: I haz the power, 2016-11-11 07:13:15]

Patchset 4 (id:??) landed as
https://crrev.com/16700579f33d11511d05966bc1c42f0b90bacd8d
Cr-Commit-Position: refs/heads/master@{#431514}