Add flag to launch two displays for mus+ash.

Add flag to launch two displays for mus+ash.

Add --multi-display flag to launch two displays when running mus+ash
under Ozone X11. This creates two 800x800 displays that are side by
side. The displays should appear fine but event targeting (and other
things) are broken. This patch is intended to allow testing and fix bugs
related to multiple displays.

Ideally, mash should use the same flags to setup multiple displays for
testing that ash contains right now. The code to parse the flag contents
is being refactored out of ash but cannot be used in //services/ui yet.

This patch also changes ViewportMetrics to store display bound instead
of just display size.

BUG=611475
Committed: https://crrev.com/bb8c2fedfb2bc06d3fb3666147e4d16781d9ebc5
Cr-Commit-Position: refs/heads/master@{#408183}

[kylechar, 2016-07-27 14:37:22]

kylechar@chromium.org changed reviewers:
+ sky@google.com

[kylechar, 2016-07-27 14:44:00]

kylechar@chromium.org changed reviewers:
+ sky@chromium.org
- sky@google.com

[kylechar, 2016-07-27 14:44:00]

Change to sky@chromium.org.

[sky, 2016-07-27 15:36:43]

LGTM

We may want a format similar to that uysed by DisplayManagerTestApi, but this is
ok for now.

https://codereview.chromium.org/2190633002/diff/1/services/ui/display/platfor...
File services/ui/display/platform_screen_impl_ozone.cc (right):

https://codereview.chromium.org/2190633002/diff/1/services/ui/display/platfor...
services/ui/display/platform_screen_impl_ozone.cc:80: if (displays.size() > 1)
Add {}

[kylechar, 2016-07-27 17:25:55]

Thanks!

https://codereview.chromium.org/2190633002/diff/1/services/ui/display/platfor...
File services/ui/display/platform_screen_impl_ozone.cc (right):

https://codereview.chromium.org/2190633002/diff/1/services/ui/display/platfor...
services/ui/display/platform_screen_impl_ozone.cc:80: if (displays.size() > 1)
On 2016/07/27 15:36:43, sky wrote:
> Add {}

Done.

[kylechar, 2016-07-27 17:27:01]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2016-07-27 17:27:01]

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/2190633002/#ps20001
(title: "Fix.")

[commit-bot: I haz the power, 2016-07-27 17:27:23]

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-07-27 18:15:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-27 18:16:18]

Description was changed from

==========
Add flag to launch two displays for mus+ash.

Add --multi-display flag to launch two displays when running mus+ash
under Ozone X11. This creates two 800x800 displays that are side by
side. The displays should appear fine but event targeting (and other
things) are broken. This patch is intended to allow testing and fix bugs
related to multiple displays.

Ideally, mash should use the same flags to setup multiple displays for
testing that ash contains right now. The code to parse the flag contents
is being refactored out of ash but cannot be used in //services/ui yet.

This patch also changes ViewportMetrics to store display bound instead
of just display size.

BUG=611475
==========

to

==========
Add flag to launch two displays for mus+ash.

Add --multi-display flag to launch two displays when running mus+ash
under Ozone X11. This creates two 800x800 displays that are side by
side. The displays should appear fine but event targeting (and other
things) are broken. This patch is intended to allow testing and fix bugs
related to multiple displays.

Ideally, mash should use the same flags to setup multiple displays for
testing that ash contains right now. The code to parse the flag contents
is being refactored out of ash but cannot be used in //services/ui yet.

This patch also changes ViewportMetrics to store display bound instead
of just display size.

BUG=611475

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

[commit-bot: I haz the power, 2016-07-27 18:16:19]

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