Chromium Code Reviews
DescriptionRevert of Don't apply maximized window bounds workaround if bounds aren't changed (patchset #1 id:1 of https://codereview.chromium.org/2454053004/ )
Reason for revert:
This resulted in a shortcut to move between monitors not working. See 656001.
Original issue's description:
> 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}
TBR=atimoxin@yandex-team.ru
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=659525, 655984
Committed: https://crrev.com/deb38c53aaeb0ca533daf90abe4f46536df1661b
Cr-Commit-Position: refs/heads/master@{#430314}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|
|||||||||||||||||||