New issue
Advanced search Search tips

Issue 809953 link

Starred by 1 user
Status: Fixed
Owner:
cnardi@chromium.org
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 809920



Sign in to add a comment

Change expectedComputedValue in css/css/fonts/font-variation-settings-parsing.html to match specified order

Project Member Reported by drott@chromium.org, Feb 7 2018 
MS WPT Test css/css/fonts/font-variation-settings-parsing.html

has 
            { value: "'wght' 1e3, 'slnt' -450.0e-1 ",     expectedComputedValue: "\"slnt\" -45, \"wght\" 1000",    isValid: true,  message: "Axis values in scientific form are valid" },

We should parse 'slnt' -450.0e-1 " correctly, currently it seems we reject it.

Comment 1 by cnardi@chromium.org, Apr 6 2018 

It looks like we do parse this correctly, we just serialize it the reverse of Edge (or I'd assume so, since MS contributed the test):

expected "\"slnt\" -45, \"wght\" 1000" but got "\"wght\" 1000, \"slnt\" -45"

Our method makes more sense, since the declaration was 'wght' 1e3, 'slnt' -450.0e-1.

Comment 2 by cnardi@chromium.org, Apr 21 2018

Owner: cnardi@chromium.org
Status: Fixed (was: Assigned)
Summary: Change expectedComputedValue in css/css/fonts/font-variation-settings-parsing.html to match specified order (was: Parse scientific form values for font-variation-settings axis parameters)
Fixed in https://github.com/w3c/web-platform-tests/pull/10535 which was imported in https://crrev.com/c/1022999. We already parse scientific notation; the serialization was just the reverse of what the spec says, so that was fixed up.
►

Sign in to add a comment