1508060 - [wpt-sync] Sync PR 14114 - SVG Painting: property parsing

Status: RESOLVED FIXED

(Core :: SVG, enhancement, P4)

Description

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

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

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

Eric Willigers <[e-mail]> wrote:
>  SVG Painting: property parsing
>  
>  Test that properties support the full grammar.
>  https://svgwg.org/svg2-draft/painting.html
>  
>  The tests highlight a number of spec and implementation issues:-
>  
>  Spec issues:
>  
>  Blink/Edge/Firefox/Safari do not accept percentage fill-opacity/stroke-opacity
>  https://github.com/w3c/svgwg/issues/403#issuecomment-439600770
>  
>  Browsers do not accept stroke-linejoin 'miter-clip' or 'arcs'
>  https://github.com/w3c/svgwg/issues/592
>  
>  Browsers accept numeric stroke-dashoffset and stroke-width
>  https://github.com/w3c/svgwg/issues/534
>  
>  Browser bugs:
>  
>  Blink image-rendering does not accept 'optimizespeed' or 'optimizequality'
>  https://bugs.chromium.org/p/chromium/issues/detail?id=901669
>  
>  Firefox rejects stroke-miterlimit less than 1
>  https://bugzilla.mozilla.org/show_bug.cgi?id=1508028
>  
>  Edge serializes fill/stroke url() without quoting the url string.
>  https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/16900961/
>  
>  Edge accepts misformed numbers fill-opacity: 1.; stroke-miterlimit: 1.; stroke-opacity: 1.;
>  https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/19656838/
>  
>  Edge accepts negative stroke-dasharray
>  Edge does not accept calc in stroke-dasharray
>  https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/19656979/
>  
>  Edge accepts negative stroke-width values
>  https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/19656907/
>  
>  Safari accepts negative stroke-width
>  https://bugs.webkit.org/show_bug.cgi?id=191804
>  
>  Safari serializes color-interpolation, color-rendering, image-rendering, shape-rendering, text-rendering in mixed case
>  https://bugs.webkit.org/show_bug.cgi?id=190685
>  
>  Edge serializes paint-order 'normal' as 'fill', unlike other browsers.
>  
>  Firefox does not use shortest paint-order serialization, instead it uses 2 keywords when 1 would suffice
>  
>  Browsers yet to implement properties:
>  
>  Firefox does not support color-rendering
>  
>  Edge does not support color-interpolation/color-rendering/image-rendering/shape-rendering/text-rendering
>  Note: When implementing, the keywords should serialize in lower case.
>  https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/19328562/
>  
>  Browsers yet to implement keywords:
>  
>  Blink/Edge/Safari do not accept fill/stroke 'context-fill' or 'context-stroke'
>  Firefox does support the keywords: https://jsfiddle.net/ericwilligers/8bhyafe1/

Comment 1

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

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

Comment 2

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

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

Comment 3

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

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

Comment 4

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

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

Comment 5

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

Ran 38 tests and 140 subtests
OK     : 38
PASS   : 124
FAIL   : 16

New tests that have failures or other problems:
/svg/painting/parsing/color-rendering-valid.svg
    e.style['color-rendering'] = "auto" should set the property value: FAIL
    e.style['color-rendering'] = "optimizequality" should set the property value: FAIL
    e.style['color-rendering'] = "optimizespeed" should set the property value: FAIL
/svg/painting/parsing/fill-opacity-valid.svg
    e.style['fill-opacity'] = "-100%" should set the property value: FAIL
    e.style['fill-opacity'] = "300%" should set the property value: FAIL
    e.style['fill-opacity'] = "50%" should set the property value: FAIL
/svg/painting/parsing/paint-order-valid.svg
    e.style['paint-order'] = "fill stroke" should set the property value: FAIL
    e.style['paint-order'] = "markers fill" should set the property value: FAIL
    e.style['paint-order'] = "stroke fill" should set the property value: FAIL
/svg/painting/parsing/stroke-linejoin-valid.svg
    e.style['stroke-linejoin'] = "arcs" should set the property value: FAIL
    e.style['stroke-linejoin'] = "miter-clip" should set the property value: FAIL
/svg/painting/parsing/stroke-miterlimit-valid.svg
    e.style['stroke-miterlimit'] = "0" should set the property value: FAIL
    e.style['stroke-miterlimit'] = "0.5" should set the property value: FAIL
/svg/painting/parsing/stroke-opacity-valid.svg
    e.style['stroke-opacity'] = "-100%" should set the property value: FAIL
    e.style['stroke-opacity'] = "300%" should set the property value: FAIL
    e.style['stroke-opacity'] = "50%" should set the property value: FAIL

Comment 6

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

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

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0818f88fb0cf
[wpt PR 14114] - SVG Painting: property parsing, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/dad19f506077
[wpt PR 14114] - Update wpt metadata, a=testonly

Comment 8

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/0818f88fb0cf
https://hg.mozilla.org/mozilla-central/rev/dad19f506077