1475611 - [wpt-sync] Sync PR 11964 - [css-properties-values-api] Support repeated values in Typed OM.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P4)

Description

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

Sync web-platform-tests PR 11964 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/11964
Details from upstream follow.

Anders Hartvoll Ruud <[e-mail]> wrote:
>  [css-properties-values-api] Support repeated values in Typed OM.
>  
>  This patch fixes two similar problems:
>  
>   1. Computed list-valued properties show up as a CSSStyleValue when calling
>      computedStyleMap().get(name).
>   2. Computed list-valued properties show up as a CSSStyleValue when calling
>      computedStyleMap().getAll(name).
>  
>  The fixes are also similar:
>  
>   1. StylePropertyMapReadOnly::get only checked property.IsRepeated() when
>      determining whether it should look inside a possible list. However,
>      CSSPropertyVariable is never repeated. This is because
>      CSSPropertyVariable does not represent a single, specific custom
>      property, but rather _all_ custom properties.
>  
>      For CSSPropertyVariable, if we have a CSSValueList, it means that
>      it is registered with either '<foo>+', or '<foo>#', and that makes it
>      a "list valued property" [1].
>  
>  2. Similar to (1), the value would be treated as unsupported if property.
>     IsRepeated() was false. Again, a situation of CSSPropertyVariable+
>     CSSValueList means that the property is registered as a list.
>  
>  Note1: The value for a custom property can only be a CSSValueList if the
>         runtime enabled feature CSSVariables2 is enabled. In all other
>         cases, it's a CSSCustomPropertyDeclaration. See
>         ComputedStyleCSSValueMapping::Get ('registry' is nullptr without
>         the feature).
>  
>  Note2: It might make more sense to ignore the repeatability of the
>         property, and instead just look at whether we actually have a
>         CSSValueList, but such a change would require a change of the
>         css-typed-om spec.
>  
>  [1] https://drafts.css-houdini.org/css-typed-om-1/#list-valued-properties
>      https://drafts.css-houdini.org/css-typed-om-1/#reify-stylevalue
>  
>  R=[e-mail]
>  
>  Bug: 862152
>  Change-Id: Ic854f54f14313b6b683601d0677dcc1dd4205ccd
>  Reviewed-on: https://chromium-review.googlesource.com/1136295
>  WPT-Export-Revision: e908fb57cbf092b93d7cd750401ac8aa37ff2b8e

Comment 1

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

PR 11964 applied with additional changes from upstream: 5c9c5d9154fa3d32a83d5ff08205e85dc0bd597f

Comment 2

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

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=241d953b49c3b63d314236d390e482fee17479b9

Comment 3

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=6f41bea9ec1abe24eeab595ec021ff0b7cfebf79

Comment 4

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

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=7d144d431623aa95efbbba51b50005c3347fa827

Comment 5

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

Ran 1 tests and 20 subtests
OK     : 1
FAIL   : 20

New tests that have failures or other problems:
/css/css-properties-values-api/typedom.tentative.html
    All computed values correctly reified in comma-separated list: FAIL
    All computed values correctly reified in space-separated list: FAIL
    Computed * is reified as CSSUnparsedValue: FAIL
    Computed <angle> is reified as CSSUnitValue: FAIL
    Computed <color> is reified as CSSStyleValue: FAIL
    Computed <custom-ident> is reified as CSSKeywordValue: FAIL
    Computed <image> [url] is reified as CSSImageValue: FAIL
    Computed <integer> is reified as CSSUnitValue: FAIL
    Computed <length-percentage> [%] is reified as CSSUnitValue: FAIL
    Computed <length-percentage> [px + %] is reified as CSSMathSum: FAIL
    Computed <length-percentage> [px] is reified as CSSUnitValue: FAIL
    Computed <length> is reified as CSSUnitValue: FAIL
    Computed <number> is reified as CSSUnitValue: FAIL
    Computed <percentage> is reified as CSSUnitValue: FAIL
    Computed <resolution> is reified as CSSUnitValue: FAIL
    Computed <time> is reified as CSSUnitValue: FAIL
    Computed <url> is reified as CSSStyleValue: FAIL
    Computed ident is reified as CSSKeywordValue: FAIL
    First computed value correctly reified in comma-separated list: FAIL
    First computed value correctly reified in space-separated list: FAIL

Comment 6

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=157ca7b8725fd43a909b1d8e5f03e80bf149e547

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8d96cff86763
[wpt PR 11964] - [css-properties-values-api] Support repeated values in Typed OM., a=testonly

Comment 8

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/8d96cff86763