Don't ignore size updates that have a width and height == 0.

Don't ignore size updates that have a width and height == 0.

It is perfectly legal for the Android WebView to have a zero height,
this is frequenly how a new WebView is first laid out. Rejecting the
initial (width, 0) size update causes the initial layout to be
too narrow.

BUG=344885
TBR=aelias@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=167594

[mkosiba (inactive), 2014-02-21 11:37:16]

This is a re-land of https://codereview.chromium.org/172203003/. The original CL
was reverted because the newly added test failed on Android (I only ran the
webkit_unit_tests on Linux...). On Android useWideViewport is on by default and
would expand the requested width of 640px to 980px causing the EXPECT_EQ to
fail.
To fix the issue i bumped the viewport size to 1280px.

[mnaganov (inactive), 2014-02-21 11:46:55]

On 2014/02/21 11:37:16, mkosiba wrote:
> This is a re-land of https://codereview.chromium.org/172203003/. The original
CL
> was reverted because the newly added test failed on Android (I only ran the
> webkit_unit_tests on Linux...). On Android useWideViewport is on by default
and
> would expand the requested width of 640px to 980px causing the EXPECT_EQ to
> fail.
> To fix the issue i bumped the viewport size to 1280px.

Looks good. Sorry, for some reason I was supposing that wide viewport is turned
on all platforms.

[mkosiba (inactive), 2014-02-21 12:00:22]

The CQ bit was checked by mkosiba@chromium.org

[commit-bot: I haz the power, 2014-02-21 12:00:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkosiba@chromium.org/175043002/1

[commit-bot: I haz the power, 2014-02-21 13:10:59]

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

[commit-bot: I haz the power, 2014-02-21 13:11:00]

Retried try job too often on linux_blink for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[mkosiba (inactive), 2014-02-21 13:43:41]

The CQ bit was checked by mkosiba@chromium.org

[commit-bot: I haz the power, 2014-02-21 13:43:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkosiba@chromium.org/175043002/1

[commit-bot: I haz the power, 2014-02-21 13:44:10]

Change committed as 167594