Bug 136065 - [GTK] [CSS Shapes] 23 failures on the tests imported from the W3C CSS Shapes test suite
Summary: [GTK] [CSS Shapes] 23 failures on the tests imported from the W3C CSS Shapes ...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-19 03:45 PDT by Carlos Alberto Lopez Perez
Modified: 2018-06-18 01:09 PDT (History)
10 users (show)

See Also:


Attachments
Crash log from shape-outside-circle-004.html (32.77 KB, text/plain)
2017-08-08 13:27 PDT, Charlie Turner
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2014-08-19 03:45:24 PDT
Revision r172733 <http://trac.webkit.org/r172733> imported 276 tests from the W3C CSS Shapes test suite.

There were various failures common to all WebKit ports that were reported on bug 135923, bug 135925, bug 135926, and bug 135927.

The following failures are specific to the GTK port:

Regressions: Unexpected text-only failures (8)
  css3/shapes/shape-outside/values/shape-margin-001.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-circle-004.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-circle-005.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-ellipse-004.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-ellipse-005.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-inset-003.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-polygon-004.html [ Failure ]
  css3/shapes/shape-outside/values/shape-outside-shape-arguments-000.html [ Failure ]

Regressions: Unexpected image-only failures (15)
  css3/shapes/shape-outside/shape-box/shape-outside-box-002.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/shape-box/shape-outside-box-003.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/shape-box/shape-outside-box-004.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/shape-box/shape-outside-box-006.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/shape-box/shape-outside-box-007.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/shape-box/shape-outside-box-008.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/circle/shape-outside-circle-017.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/circle/shape-outside-circle-027.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/ellipse/shape-outside-ellipse-017.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/inset/shape-outside-inset-010.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/inset/shape-outside-inset-011.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/inset/shape-outside-inset-012.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/inset/shape-outside-inset-013.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/inset/shape-outside-inset-014.html [ ImageOnlyFailure ]
  css3/shapes/shape-outside/supported-shapes/inset/shape-outside-inset-015.html [ ImageOnlyFailure ]


You can check the diffs here: http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r172733%20%282204%29/css3/shapes/shape-outside/
Comment 1 Carlos Alberto Lopez Perez 2014-08-19 07:31:34 PDT
Updated the GTK TestExpectations for this failures on http://trac.webkit.org/changeset/172750
Comment 2 Carlos Garcia Campos 2017-02-10 02:51:24 PST
It seems some of these tests are duplicated in imported/w3c/csswg-test/css-shapes-1/shape-outside. 

imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-margin-001.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-circle-004.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-circle-005.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-ellipse-004.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-ellipse-005.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-inset-003.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-polygon-004.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/values/shape-outside-shape-arguments-000.html [ Failure ]
imported/w3c/csswg-test/css-shapes-1/shape-outside/supported-shapes/circle/shape-outside-circle-027.html [ ImageOnlyFailure ]

All these are also failing in blink, see crbug.com/441840 and crbug.com/443615
Comment 3 Charlie Turner 2017-08-08 13:27:16 PDT
Created attachment 317604 [details]
Crash log from  shape-outside-circle-004.html

Starting seeing stack corruption in these tests after r220352,

  css3/shapes/shape-outside/values/shape-outside-circle-004.html [ Crash ]
  css3/shapes/shape-outside/values/shape-outside-ellipse-004.html [ Crash ]

The crash log isn't terribly useful, because the stack is corrupted. Last signs of life are around JSDOMWindowBase::printErrorMessage.
Comment 4 Charlie Turner 2017-08-08 13:28:57 PDT
CC'd fpizlo on this one just in case there have been any other sightings of stack corruption with r220352
Comment 5 Charlie Turner 2017-08-08 13:31:24 PDT
The stack corruption is also non-deterministic, for a little extra spice.
Comment 6 Charlie Turner 2017-08-08 13:50:41 PDT
This was fixed right after I noticed it in https://trac.webkit.org/changeset/220416/webkit, very nice :)
Comment 7 Michael Catanzaro 2018-06-17 12:27:35 PDT
imported/w3c/web-platform-tests/css/css-shapes/shape-outside/supported-shapes/circle/shape-outside-circle-027.html was fixed by r232903.

css3/shapes/shape-outside/supported-shapes/circle/shape-outside-circle-027.html is still broken. Looks like that's the last affected test here.