Mus: Remove CompositorFrameSinkType

Mus: Remove CompositorFrameSinkType

Now that underlay surfaces are gone, there is no reason to have a
CompositorFrameSinkType in Mus.

BUG=672943
Review-Url: https://codereview.chromium.org/2617403003
Cr-Commit-Position: refs/heads/master@{#442675}
Committed: https://chromium.googlesource.com/chromium/src/+/ec78ec0763a819fc5d5f346bef3538ff090368a5

[Fady Samuel, 2017-01-09 16:49:46]

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

[commit-bot: I haz the power, 2017-01-09 16:50:13]

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, 2017-01-09 17:34:29]

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

[commit-bot: I haz the power, 2017-01-09 17:34:30]

Dry run: 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_...)

[Fady Samuel, 2017-01-09 17:41:46]

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

[commit-bot: I haz the power, 2017-01-09 17:42:09]

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

[Fady Samuel, 2017-01-09 17:50:16]

Description was changed from

==========
Mus: Remoive CompositorFrameSinkType

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

to

==========
Mus: Remove CompositorFrameSinkType

Now that underlay surfaces are gone, there is no reason to have a
CompositorFrameSinkType in Mus.

BUG=672943
==========

[Fady Samuel, 2017-01-09 17:50:30]

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

[Fady Samuel, 2017-01-09 17:50:31]

PTAL Scott!

[commit-bot: I haz the power, 2017-01-09 18:31:10]

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

[commit-bot: I haz the power, 2017-01-09 18:31:11]

Dry run: 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_...)

[sky, 2017-01-09 19:03:00]

LGTM

https://codereview.chromium.org/2617403003/diff/20001/services/ui/ws/server_w...
File services/ui/ws/server_window_compositor_frame_sink_manager.h (right):

https://codereview.chromium.org/2617403003/diff/20001/services/ui/ws/server_w...
services/ui/ws/server_window_compositor_frame_sink_manager.h:8: #include <map>
remove include?

[Fady Samuel, 2017-01-09 19:27:57]

Thanks, CQ'ing.

https://codereview.chromium.org/2617403003/diff/20001/services/ui/ws/server_w...
File services/ui/ws/server_window_compositor_frame_sink_manager.h (right):

https://codereview.chromium.org/2617403003/diff/20001/services/ui/ws/server_w...
services/ui/ws/server_window_compositor_frame_sink_manager.h:8: #include <map>
On 2017/01/09 19:03:00, sky wrote:
> remove include?

Done.

[Fady Samuel, 2017-01-09 19:28:03]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2017-01-09 19:28:03]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2617403003/#ps40001
(title: "Remove unnecessary include")

[commit-bot: I haz the power, 2017-01-09 19:28:41]

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, 2017-01-09 20:20:14]

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

[commit-bot: I haz the power, 2017-01-09 20:20:15]

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

[Fady Samuel, 2017-01-10 18:45:44]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2017-01-10 18:45:45]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2617403003/#ps60001
(title: "Remove unnecessary include and rebase")

[commit-bot: I haz the power, 2017-01-10 18:46:13]

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

[Fady Samuel, 2017-01-10 19:05:20]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2017-01-10 19:05:20]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2617403003/#ps80001
(title: "Fix some unit tests")

[commit-bot: I haz the power, 2017-01-10 19:05:49]

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, 2017-01-10 20:35:59]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1484075120620440,
"parent_rev": "c3a36842e3a92c456fd2c9c2e99f8da6f1cde2f8", "commit_rev":
"ec78ec0763a819fc5d5f346bef3538ff090368a5"}

[commit-bot: I haz the power, 2017-01-10 20:36:33]

Description was changed from

==========
Mus: Remove CompositorFrameSinkType

Now that underlay surfaces are gone, there is no reason to have a
CompositorFrameSinkType in Mus.

BUG=672943
==========

to

==========
Mus: Remove CompositorFrameSinkType

Now that underlay surfaces are gone, there is no reason to have a
CompositorFrameSinkType in Mus.

BUG=672943

Review-Url: https://codereview.chromium.org/2617403003
Cr-Commit-Position: refs/heads/master@{#442675}
Committed:
https://chromium.googlesource.com/chromium/src/+/ec78ec0763a819fc5d5f346bef35...
==========

[commit-bot: I haz the power, 2017-01-10 20:36:36]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/ec78ec0763a819fc5d5f346bef35...

[Wez, 2017-02-07 02:29:59]

wez@chromium.org changed reviewers:
+ tsepez@chromium.org, wez@chromium.org

[Wez, 2017-02-07 02:30:00]

Hallo tsepez@chromium.org!
Due to a depot_tools patch which mistakenly removed the OWNERS check for
non-source files (see crbug.com/684270), the following files landed in this CL
and need a retrospective review from you:
	services/ui/public/interfaces/window_tree.mojom
	services/ui/public/interfaces/window_tree_constants.mojom
Thanks,
Wez