Mus: ws/ should not depend on surfaces/

Mus: ws/ should not depend on surfaces/

As part of the mus-ws/mus-gpu split, the ws/ directory should not depend on
anything in surfaces/.

BUG=none

Committed: https://crrev.com/66f5a3fc0ebcf1fc89665685444eda4c1855eae2
Cr-Commit-Position: refs/heads/master@{#434095}

[Fady Samuel, 2016-11-22 22:54:57]

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

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

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

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

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

[Fady Samuel, 2016-11-22 22:57:01]

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

[Fady Samuel, 2016-11-22 22:57:01]

PTAL sky@

[Fady Samuel, 2016-11-22 22:57:01]

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

[Fady Samuel, 2016-11-22 22:57:01]

PTAL sky@

[Fady Samuel, 2016-11-22 22:57:02]

PTAL sky@

[sky, 2016-11-22 23:03:01]

LGTM

[commit-bot: I haz the power, 2016-11-23 00:38:22]

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

[commit-bot: I haz the power, 2016-11-23 00:38:23]

Dry run: This issue passed the CQ dry run.

[Fady Samuel, 2016-11-23 00:40:58]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2016-11-23 00:41:50]

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-23 01:16:12]

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

[commit-bot: I haz the power, 2016-11-23 01:16:14]

Failed to apply patch for services/ui/ws/platform_display_init_params.cc:
While running git apply --index -p1;
  error: patch failed: services/ui/ws/platform_display_init_params.cc:4
  error: services/ui/ws/platform_display_init_params.cc: patch does not apply

Patch:       services/ui/ws/platform_display_init_params.cc
Index: services/ui/ws/platform_display_init_params.cc
diff --git a/services/ui/ws/platform_display_init_params.cc
b/services/ui/ws/platform_display_init_params.cc
index
213483866944750377d5f8b4f97fd7e713400006..0c698a9f4dcc74aaa135a82d7a34b18c6bebcd70
100644
--- a/services/ui/ws/platform_display_init_params.cc
+++ b/services/ui/ws/platform_display_init_params.cc
@@ -4,7 +4,6 @@
 
 #include "services/ui/ws/platform_display_init_params.h"
 
-#include "services/ui/surfaces/display_compositor.h"
 #include "services/ui/ws/server_window.h"
 #include "ui/display/display.h"

[Fady Samuel, 2016-11-23 01:32:48]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-11-23 01:32:48]

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/2521123004/#ps20001
(title: "Rebased")

[commit-bot: I haz the power, 2016-11-23 01:33:15]

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-23 02:56:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1479864768408970,
"parent_rev": "16ea9266923d89065de678becc7f8be46c3da044", "commit_rev":
"9fa735257a4439d8767cdd274f6a937a62082d59"}

[commit-bot: I haz the power, 2016-11-23 02:56:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-23 02:59:54]

Description was changed from

==========
Mus: ws/ should not depend on surfaces/

As part of the mus-ws/mus-gpu split, the ws/ directory should not depend on
anything in surfaces/.

BUG=none
==========

to

==========
Mus: ws/ should not depend on surfaces/

As part of the mus-ws/mus-gpu split, the ws/ directory should not depend on
anything in surfaces/.

BUG=none

Committed: https://crrev.com/66f5a3fc0ebcf1fc89665685444eda4c1855eae2
Cr-Commit-Position: refs/heads/master@{#434095}
==========

[commit-bot: I haz the power, 2016-11-23 02:59:55]

Patchset 2 (id:??) landed as
https://crrev.com/66f5a3fc0ebcf1fc89665685444eda4c1855eae2
Cr-Commit-Position: refs/heads/master@{#434095}

[nhiroki, 2016-11-24 03:18:16]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2527003002/ by nhiroki@chromium.org.

The reason for reverting is: Speculative revert for issue 668105. This seems to
be failing WindowServerTest.* and WindowTreeClientTest (mus_ws_unittests) on
Ubuntu-12.04 and Windows-7-SP1..