exo: Confine windows to primary display

exo: Confine windows to primary display

Multi-display support in ARC was punted to the next milestone. This CL
temporarily prevents windows from being dragged to secondary displays
by using display rather than screen coordinates for window bounds.

BUG=714832
TEST=ARC apps cannot be dragged to secondary displays.

Review-Url: https://codereview.chromium.org/2860863004
Cr-Commit-Position: refs/heads/master@{#469514}
Committed: https://chromium.googlesource.com/chromium/src/+/53f36cb4e8b4301b733e64d392769ee3ab9a5ca8

[Dominik Laskowski, 2017-05-04 21:10:04]

domlaskowski@chromium.org changed reviewers:
+ reveman@chromium.org

[Dominik Laskowski, 2017-05-04 21:10:04]

PTAL.

[reveman, 2017-05-04 21:11:46]

This affects non-arc windows too, right? Is that what we want?

[Dominik Laskowski, 2017-05-04 21:54:38]

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

[commit-bot: I haz the power, 2017-05-04 21:55:01]

Dry run: CQ is trying da patch.

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

[reveman, 2017-05-04 21:56:11]

lgtm with nit

https://codereview.chromium.org/2860863004/diff/1/components/exo/wm_helper_as...
File components/exo/wm_helper_ash.cc (right):

https://codereview.chromium.org/2860863004/diff/1/components/exo/wm_helper_as...
components/exo/wm_helper_ash.cc:63: return
ash::Shell::GetContainer(ash::Shell::GetPrimaryRootWindow(),
nit: please add a TODO with link to bug here to explain that this prevents all
exo clients from using multiple displays

[commit-bot: I haz the power, 2017-05-04 22:26:46]

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

[commit-bot: I haz the power, 2017-05-04 22:26:47]

Dry run: This issue passed the CQ dry run.

[Dominik Laskowski, 2017-05-04 22:42:09]

https://codereview.chromium.org/2860863004/diff/1/components/exo/wm_helper_as...
File components/exo/wm_helper_ash.cc (right):

https://codereview.chromium.org/2860863004/diff/1/components/exo/wm_helper_as...
components/exo/wm_helper_ash.cc:63: return
ash::Shell::GetContainer(ash::Shell::GetPrimaryRootWindow(),
On 2017/05/04 21:56:11, reveman wrote:
> nit: please add a TODO with link to bug here to explain that this prevents all
> exo clients from using multiple displays

Done.

[Dominik Laskowski, 2017-05-04 22:43:24]

The CQ bit was checked by domlaskowski@chromium.org

[Dominik Laskowski, 2017-05-04 22:43:24]

The patchset sent to the CQ was uploaded after l-g-t-m from reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2860863004/#ps20001
(title: "Add TODOs")

[commit-bot: I haz the power, 2017-05-04 22:44:38]

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-05-04 23:24:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493937804387240,
"parent_rev": "beabb626561e9899cb3099b3c7cd5a1447bf00f7", "commit_rev":
"53f36cb4e8b4301b733e64d392769ee3ab9a5ca8"}

[commit-bot: I haz the power, 2017-05-04 23:25:02]

Description was changed from

==========
exo: Confine windows to primary display

Multi-display support in ARC was punted to the next milestone. This CL
temporarily prevents windows from being dragged to secondary displays
by using display rather than screen coordinates for window bounds.

BUG=714832
TEST=ARC apps cannot be dragged to secondary displays.
==========

to

==========
exo: Confine windows to primary display

Multi-display support in ARC was punted to the next milestone. This CL
temporarily prevents windows from being dragged to secondary displays
by using display rather than screen coordinates for window bounds.

BUG=714832
TEST=ARC apps cannot be dragged to secondary displays.

Review-Url: https://codereview.chromium.org/2860863004
Cr-Commit-Position: refs/heads/master@{#469514}
Committed:
https://chromium.googlesource.com/chromium/src/+/53f36cb4e8b4301b733e64d39276...
==========

[commit-bot: I haz the power, 2017-05-04 23:25:03]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/53f36cb4e8b4301b733e64d39276...