Clamp the restore bounds to new workspace when a display is disconnected.

Clamp the restore bounds to new workspace when a display is disconnected.

Reparent does adjust the bounds, but when it's partially offline, i can
push the window further to the different direction if the destination
display is larger.

BUG=659925
TEST=Updated the test to cover the scenario.

Review-Url: https://codereview.chromium.org/2703333003
Cr-Commit-Position: refs/heads/master@{#451899}
Committed: https://chromium.googlesource.com/chromium/src/+/92224ac504e06d16d48ceeb63eb3160519629ffc

[oshima, 2017-02-21 08:25:07]

Patchset #2 (id:20001) has been deleted

[oshima, 2017-02-21 08:25:19]

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

[commit-bot: I haz the power, 2017-02-21 08:25:31]

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

[oshima, 2017-02-21 08:27:22]

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

[oshima, 2017-02-21 08:29:40]

Patchset #2 (id:40001) has been deleted

[oshima, 2017-02-21 08:32:44]

Description was changed from

==========
Clamp the restore bounds to new workspace when a display is disconnected.

BUG=659925
TEST=Updated the test to cover the scenario.
==========

to

==========
Clamp the restore bounds to new workspace when a display is disconnected.

Reparent does adjust the bounds, but when it's partially offline, i can
push the window further to the different direction if the destination
display is larger.

BUG=659925
TEST=Updated the test to cover the scenario.
==========

[afakhry, 2017-02-21 17:42:51]

https://codereview.chromium.org/2703333003/diff/60001/ash/root_window_control...
File ash/root_window_controller_unittest.cc (right):

https://codereview.chromium.org/2703333003/diff/60001/ash/root_window_control...
ash/root_window_controller_unittest.cc:177:
fullscreen->GetNativeView()->GetBoundsInRootWindow().ToString());
These EXPECT_EQ()s seem to be the exact same as the ones above.

[oshima, 2017-02-21 23:54:55]

https://codereview.chromium.org/2703333003/diff/60001/ash/root_window_control...
File ash/root_window_controller_unittest.cc (right):

https://codereview.chromium.org/2703333003/diff/60001/ash/root_window_control...
ash/root_window_controller_unittest.cc:177:
fullscreen->GetNativeView()->GetBoundsInRootWindow().ToString());
On 2017/02/21 17:42:51, afakhry wrote:
> These EXPECT_EQ()s seem to be the exact same as the ones above.

oops, not sure why I added. removed.

[afakhry, 2017-02-21 23:56:09]

lgtm

[oshima, 2017-02-22 04:50:38]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2017-02-22 04:51:37]

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-02-22 06:27:50]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1487739038651060,
"parent_rev": "7aa56504f87294f9b5b047401eb6fe4b59768c9d", "commit_rev":
"92224ac504e06d16d48ceeb63eb3160519629ffc"}

[commit-bot: I haz the power, 2017-02-22 06:28:38]

Description was changed from

==========
Clamp the restore bounds to new workspace when a display is disconnected.

Reparent does adjust the bounds, but when it's partially offline, i can
push the window further to the different direction if the destination
display is larger.

BUG=659925
TEST=Updated the test to cover the scenario.
==========

to

==========
Clamp the restore bounds to new workspace when a display is disconnected.

Reparent does adjust the bounds, but when it's partially offline, i can
push the window further to the different direction if the destination
display is larger.

BUG=659925
TEST=Updated the test to cover the scenario.

Review-Url: https://codereview.chromium.org/2703333003
Cr-Commit-Position: refs/heads/master@{#451899}
Committed:
https://chromium.googlesource.com/chromium/src/+/92224ac504e06d16d48ceeb63eb3...
==========

[commit-bot: I haz the power, 2017-02-22 06:28:39]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/92224ac504e06d16d48ceeb63eb3...