[CSSMQ] Use initial values as base for relative units.

[CSSMQ] Use initial values as base for relative units.

Don't use computed style for root element as base for relative lengths. The
media queries spec says:

"Relative units in media queries are based on the initial value, which means
that units are never based on results of declarations."

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

[kenneth.r.christiansen, 2013-08-07 10:49:28]

https://codereview.chromium.org/22258007/diff/1/Source/core/css/MediaQueryEva...
File Source/core/css/MediaQueryEvaluator.cpp (right):

https://codereview.chromium.org/22258007/diff/1/Source/core/css/MediaQueryEva...
Source/core/css/MediaQueryEvaluator.cpp:353: result =
primitiveValue->computeLength<int>(style, style, 1.0 /* multiplier */, true /*
computingFontSize */);
Could we add some /* */ to know what it means, especially given style x2

[rune, 2013-08-07 11:46:08]

https://codereview.chromium.org/22258007/diff/1/Source/core/css/MediaQueryEva...
File Source/core/css/MediaQueryEvaluator.cpp (right):

https://codereview.chromium.org/22258007/diff/1/Source/core/css/MediaQueryEva...
Source/core/css/MediaQueryEvaluator.cpp:353: result =
primitiveValue->computeLength<int>(style, style, 1.0 /* multiplier */, true /*
computingFontSize */);
On 2013/08/07 10:49:28, kenneth.r.christiansen wrote:
> Could we add some /* */ to know what it means, especially given style x2

Done.

[apavlov, 2013-08-07 12:46:35]

lgtm.

Could you please re-upload the patch using

git cl upload --no-find-copies

first? Since the test .html has not actually been copied (but is displayed as
such in the review tool)?

[commit-bot: I haz the power, 2013-08-07 12:56:42]

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

[rune, 2013-08-07 12:57:09]

On 2013/08/07 12:46:35, apavlov wrote:
> lgtm.
> 
> Could you please re-upload the patch using
> 
> git cl upload --no-find-copies
> 
> first? Since the test .html has not actually been copied (but is displayed as
> such in the review tool)?

Done. Thanks for the review.

[commit-bot: I haz the power, 2013-08-07 14:31:10]

Change committed as 155692