1840477 - [css-properties-values-api] Finish syntax validation

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task)

Description

[Emilio Cobos Álvarez (:emilio)]

Another decent chunk of work for @property / registerProperty is, given a syntax Descriptor and an UnparsedValue, parse the later following the syntax rules of the former.

Once we parse those, probably into something like:

enum Value {
    Number(specified::Number),
    Length(specified::Length),
    Transform(...),
    Url(...),
    // etc
}

We should also add checks for computational independence. That should make @property validation and registerProperty follow the spec.

See the references to this bug in the codebase.

Comment 1

[Zach Hoffman [:zrhoffman]]

Attachment: Bug 1840477 - Validate initial value syntax r=#style

With this patch, all the WPTs for initial value validation pass, except
for one whose syntax fails to parse (see bug 1843988).

Depends on D183806

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2664ab12c402
Validate initial value syntax r=emilio

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41122 for changes under testing/web-platform/tests

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/2664ab12c402

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot