Don't apply maximized window bounds workaround if bounds aren't changed

Don't apply maximized window bounds workaround if bounds aren't changed

In HWNDMessageHandler::OnWindowPosChanging() was added code that
restores correct maximized window's bounds after attaching or detaching
additional display (see crrev.com/6bb7f8192). But this workaround also
applied if OnWindowPosChanging() was called with SWP_NOMOVE/SWP_NOSIZE
flags that leads to bugs such as crbug.com/659525 and crbug.com/655984.

This CL adds additional check for SWP_NOMOVE/SWP_NOSIZE flags before
applying workaround and fixes this bugs.

BUG=659525, 655984
Committed: https://crrev.com/6279588b7ebc740f85409107dd91b5c3ce628065
Cr-Commit-Position: refs/heads/master@{#428368}

[atimoxin, 2016-10-28 11:47:40]

atimoxin@yandex-team.ru changed reviewers:
+ sky@chromium.org

[sky, 2016-10-28 13:36:47]

LGTM

[atimoxin, 2016-10-28 13:45:47]

The CQ bit was checked by atimoxin@yandex-team.ru

[commit-bot: I haz the power, 2016-10-28 13:45:58]

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-10-28 14:52:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-28 14:55:02]

Description was changed from

==========
Don't apply maximized window bounds workaround if bounds aren't changed

In HWNDMessageHandler::OnWindowPosChanging() was added code that
restores correct maximized window's bounds after attaching or detaching
additional display (see crrev.com/6bb7f8192). But this workaround also
applied if OnWindowPosChanging() was called with SWP_NOMOVE/SWP_NOSIZE
flags that leads to bugs such as crbug.com/659525 and crbug.com/655984.

This CL adds additional check for SWP_NOMOVE/SWP_NOSIZE flags before
applying workaround and fixes this bugs.

BUG=659525,655984
==========

to

==========
Don't apply maximized window bounds workaround if bounds aren't changed

In HWNDMessageHandler::OnWindowPosChanging() was added code that
restores correct maximized window's bounds after attaching or detaching
additional display (see crrev.com/6bb7f8192). But this workaround also
applied if OnWindowPosChanging() was called with SWP_NOMOVE/SWP_NOSIZE
flags that leads to bugs such as crbug.com/659525 and crbug.com/655984.

This CL adds additional check for SWP_NOMOVE/SWP_NOSIZE flags before
applying workaround and fixes this bugs.

BUG=659525,655984

Committed: https://crrev.com/6279588b7ebc740f85409107dd91b5c3ce628065
Cr-Commit-Position: refs/heads/master@{#428368}
==========

[commit-bot: I haz the power, 2016-10-28 14:55:03]

Patchset 1 (id:??) landed as
https://crrev.com/6279588b7ebc740f85409107dd91b5c3ce628065
Cr-Commit-Position: refs/heads/master@{#428368}

[sky, 2016-11-07 17:17:36]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2482933002/ by sky@chromium.org.

The reason for reverting is: This resulted in a shortcut to move between
monitors not working. See 656001..