Recompute percentage based @viewport on resize.

Recompute percentage based @viewport on resize.

Percentage values were resolved to px lengths at cascade time for @viewport
widths and heights. This change keeps them as percentage values and recompute
the actual viewport from the initial viewport and the percentage value when
the intial viewport changes (window resize). Likewise for viewport relative
units (vw, vh, vmin, vmax).

This fixes constrain-006 and constrain-007 in the w3c testsuite.

This CL also disallows 'initial' and 'inherit' as descriptor values and returns 'auto' for unknown non-primitive values to avoid crashers and improve on the original CL for the same bug: https://codereview.chromium.org/22549002/

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

[rune, 2013-09-03 22:14:22]

Patch Set 1 should be identical to the reverted CL. Patch Set 2 contains the
fix.

[kenneth.r.christiansen, 2013-09-04 10:57:51]

lgtm

[darktears, 2013-09-04 11:05:52]

On 2013/09/04 10:57:51, kenneth.r.christiansen wrote:
> lgtm

lgtm

[esprehn, 2013-09-04 11:22:24]

btw you can't TBR functional changes like this. I think you wanted R=.

https://codereview.chromium.org/23604037/diff/3001/Source/core/css/resolver/V...
File Source/core/css/resolver/ViewportStyleResolver.cpp (right):

https://codereview.chromium.org/23604037/diff/3001/Source/core/css/resolver/V...
Source/core/css/resolver/ViewportStyleResolver.cpp:164: Length
ViewportStyleResolver::getViewportLengthValue(CSSPropertyID id) const
This should really be called viewportLengthValue() or just viewportLength(). We
don't prefix things with get like this.

[rune, 2013-09-04 14:53:31]

https://codereview.chromium.org/23604037/diff/3001/Source/core/css/resolver/V...
File Source/core/css/resolver/ViewportStyleResolver.cpp (right):

https://codereview.chromium.org/23604037/diff/3001/Source/core/css/resolver/V...
Source/core/css/resolver/ViewportStyleResolver.cpp:164: Length
ViewportStyleResolver::getViewportLengthValue(CSSPropertyID id) const
On 2013/09/04 11:22:24, esprehn wrote:
> This should really be called viewportLengthValue() or just viewportLength().
We
> don't prefix things with get like this.

Done.

[commit-bot: I haz the power, 2013-09-04 14:58:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/23604037/13001

[commit-bot: I haz the power, 2013-09-04 17:45:17]

Change committed as 157225