[Win] Update HWND max size if either width or height is set.

[Win] Update HWND max size if either width or height is set.

gfx::Size::IsEmpty() is equivalent to
(width() == 0 || height() == 0), so the max size is not
correctly set when only one dimension is provided.

BUG=0
Committed: https://crrev.com/1a13cda009929d61b4cc859ddb4f510307b7cad6
Cr-Commit-Position: refs/heads/master@{#299625}

[jackhou1, 2014-10-14 04:45:40]

jackhou@chromium.org changed reviewers:
+ sky@chromium.org

[jackhou1, 2014-10-14 04:45:40]

sky, could you take a look?

[sky, 2014-10-14 19:20:11]

LGTM

https://codereview.chromium.org/656723002/diff/1/ui/views/win/hwnd_message_ha...
File ui/views/win/hwnd_message_handler.cc (right):

https://codereview.chromium.org/656723002/diff/1/ui/views/win/hwnd_message_ha...
ui/views/win/hwnd_message_handler.cc:1434: if (max_window_size.width()) {
nit: no {} here and 1437. Also, it's worth documenting this (that is, a comment
right here that either axis may be 0).

[jackhou1, 2014-10-14 23:49:24]

https://codereview.chromium.org/656723002/diff/1/ui/views/win/hwnd_message_ha...
File ui/views/win/hwnd_message_handler.cc (right):

https://codereview.chromium.org/656723002/diff/1/ui/views/win/hwnd_message_ha...
ui/views/win/hwnd_message_handler.cc:1434: if (max_window_size.width()) {
On 2014/10/14 19:20:10, sky wrote:
> nit: no {} here and 1437. Also, it's worth documenting this (that is, a
comment
> right here that either axis may be 0).

Done.

[jackhou1, 2014-10-14 23:49:27]

The CQ bit was checked by jackhou@chromium.org

[commit-bot: I haz the power, 2014-10-14 23:50:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/656723002/20001

[commit-bot: I haz the power, 2014-10-15 02:17:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-15 02:18:15]

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