1887263 - Add a wpt test for an invalid path that other browsers incorrectly allow.

Status: RESOLVED FIXED

(Core :: Layout, defect, P3)

Description

[Thomas Wisniewski [:twisniewski]]

See https://wpt.fyi/results/css/css-masking/clip-path/clip-path-path-with-zoom-hittest.html

Chrome and Safari pass.

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Boris, we fail to parse the path in here: https://searchfox.org/mozilla-central/rev/2537e5eaafaab9a7ef6be5cfc8e9b1e2c747fdfd/testing/web-platform/tests/css/css-masking/clip-path/clip-path-path-with-zoom-hittest.html#13

Is that a bug in the test or in our implementation?

Comment 2

[Robert Longson [:longsonr]]

Bug in the test, commas are only valid between pairs of numbers. Safari and Chrome have been known failures for decades. Other implementations such as Batik are in our camp (as is the specification, of course).

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1887263 - Add a wpt test for an invalid path that other browsers incorrectly allow. r=longsonr,boris,#style

Commas are only valid between pairs of numbers as per the SVG spec.

Comment 4

[Emilio Cobos Álvarez (:emilio)]

Given I fixed this test and we now pass it let's repurpose this.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8ef786595aad
Add a wpt test for an invalid path that other browsers incorrectly allow. r=longsonr

Comment 6

[Iulian Moraru]

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

Comment 7

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

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

Comment 8

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

Upstream PR merged by moz-wptsync-bot