Correctly restore fullscreen window bounds when leaving overview mode.

Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489
Committed: https://crrev.com/52d3a47c75a47c5e8b1e424d53b48401cd50dbdd
Cr-Commit-Position: refs/heads/master@{#410393}

[hariank, 2016-08-05 22:17:30]

Description was changed from

==========
Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489
==========

to

==========
Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489
==========

[hariank, 2016-08-05 22:17:30]

hariank@google.com changed reviewers:
+ oshima@chromium.org, pkotwicz@chromium.org, reveman@chromium.org

[hariank, 2016-08-05 22:18:00]

Hi, please review this small change. Thanks!

[hariank, 2016-08-05 22:18:16]

The CQ bit was checked by hariank@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-05 22:18:43]

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, 2016-08-05 23:02:18]

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

[commit-bot: I haz the power, 2016-08-05 23:02:20]

Dry run: This issue passed the CQ dry run.

[oshima, 2016-08-05 23:18:00]

lgtm with a nit

https://codereview.chromium.org/2225523002/diff/1/ash/common/wm/overview/wind...
File ash/common/wm/overview/window_selector.cc (right):

https://codereview.chromium.org/2225523002/diff/1/ash/common/wm/overview/wind...
ash/common/wm/overview/window_selector.cc:556: // If only the work area changes,
do not position windows in overview.
... area changes, there is no need to reposition windows in overview.

[reveman, 2016-08-08 16:41:28]

lgtm

[hariank, 2016-08-08 17:12:57]

hariank@google.com changed reviewers:
- pkotwicz@chromium.org

[hariank, 2016-08-08 17:13:02]

The CQ bit was checked by hariank@google.com

[hariank, 2016-08-08 17:13:02]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2225523002/#ps20001
(title: "Correctly restore fullscreen window bounds when leaving overview
mode.")

[commit-bot: I haz the power, 2016-08-08 17:13:31]

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-08-08 18:05:04]

Description was changed from

==========
Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489
==========

to

==========
Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489
==========

[commit-bot: I haz the power, 2016-08-08 18:05:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-08 18:07:50]

Description was changed from

==========
Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489
==========

to

==========
Correctly restore fullscreen window bounds when leaving overview mode.

In WindowSelector's OnDisplayMetricsChanged, reposition windows only if
a metric other than the work area is changed.

BUG=634489

Committed: https://crrev.com/52d3a47c75a47c5e8b1e424d53b48401cd50dbdd
Cr-Commit-Position: refs/heads/master@{#410393}
==========

[commit-bot: I haz the power, 2016-08-08 18:07:50]

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