Allow unified desktop for 3+ displays

Allow unified desktop for 3+ displays

BUG=599232
Committed: https://crrev.com/99029a1c9485daea9d1c642b6a733c81f22867a4
Cr-Commit-Position: refs/heads/master@{#384140}

[oshima, 2016-03-30 20:55:44]

Patchset #1 (id:1) has been deleted

[oshima, 2016-03-30 20:55:53]

Description was changed from

==========
Allow unified desktop for 3+ displays

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

to

==========
Allow unified desktop for 3+ displays

BUG=599232
==========

[oshima, 2016-03-30 20:55:54]

oshima@chromium.org changed reviewers:
+ stevenjb@chromium.org

[oshima, 2016-03-30 20:59:00]

Patchset #1 (id:20001) has been deleted

[oshima, 2016-03-30 21:00:05]

https://codereview.chromium.org/1845723002/diff/40001/ash/display/display_man...
File ash/display/display_manager.cc (right):

https://codereview.chromium.org/1845723002/diff/40001/ash/display/display_man...
ash/display/display_manager.cc:1087: return;
This CL just moved this change from 1083. The rest is just indent change.

[oshima, 2016-03-30 21:00:11]

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

[commit-bot: I haz the power, 2016-03-30 21:00:38]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845723002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845723002/40001

[stevenjb, 2016-03-30 21:09:02]

lgtm

[commit-bot: I haz the power, 2016-03-30 21:38:52]

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

[commit-bot: I haz the power, 2016-03-30 21:38:53]

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

[oshima, 2016-03-30 22:01:32]

PTAL

https://codereview.chromium.org/1845723002/diff/40001/ash/display/display_man...
File ash/display/display_manager.cc (right):

https://codereview.chromium.org/1845723002/diff/40001/ash/display/display_man...
ash/display/display_manager.cc:1085: DCHECK_EQ(2u, display_info_list->size());
This check does not work when hard ware mirroring is used. Removed.

I also had to add num dislpay == 1 check for Unified because
"multi_display_mode_" stays unchanged for single case.

[stevenjb, 2016-03-30 22:14:17]

lgtm

[oshima, 2016-03-30 22:19:03]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2016-03-30 22:19:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845723002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845723002/60001

[commit-bot: I haz the power, 2016-03-30 23:14:41]

Description was changed from

==========
Allow unified desktop for 3+ displays

BUG=599232
==========

to

==========
Allow unified desktop for 3+ displays

BUG=599232
==========

[commit-bot: I haz the power, 2016-03-30 23:14:42]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-03-30 23:17:39]

Description was changed from

==========
Allow unified desktop for 3+ displays

BUG=599232
==========

to

==========
Allow unified desktop for 3+ displays

BUG=599232

Committed: https://crrev.com/99029a1c9485daea9d1c642b6a733c81f22867a4
Cr-Commit-Position: refs/heads/master@{#384140}
==========

[commit-bot: I haz the power, 2016-03-30 23:17:40]

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