Bug 48987 – [Qt] svg/animations/animate-path-nested-transforms.html fails

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: New Bugs

[Csaba Osztrogonác]

svg/animations/animate-path-nested-transforms.html introduced in r71314 and fails:
http://build.webkit.org/results/Qt%20Linux%20Release/r71319%20%2823154%29/svg/animations/animate-path-nested-transforms-pretty-diff.html

[Csaba Osztrogonác]

I added svg/animations/animate-path-nested-transforms.html to
the Skipped list: http://trac.webkit.org/changeset/71320

[Shane Stephens]

I'd like to help fix this but I don't know much about the Qt port.  I can tell you about the patch that introduced the test however.

r71314 was introduced to fix an issue in the way animated path transforms interact with container transforms.  The test (svg/animations/animate-path-nested-transforms.html) fails with the old behaviour, and (at least in webkit main) passes with the new behaviour.

The test looks at the starting and ending bounding box of an animated square, to make sure that the square moves overall horizontally more than vertically.

Looking at the output of the test for the Qt port, it seems that the animation proceeds correctly, but that the start and ending locations are slightly incorrect.  I am unable to run this test on a linux machine (at least not for another week) to confirm this, but it almost seems as if the bounding box is larger when the test is run using Qt.  If this is the case, then this specific test can be fixed by providing expected results for Qt that roughly match the output (e.g. (360, 270) and (700, 230)); however I'm also quite curious as to why the values are so different.

[Tony Chang]

Adding some people who might know about svg in qt.

[Philippe Normand]

New test crashes on GTK, see bug 49065

[Shane Stephens]

The crashing in debug mode is being tracked in https://bugs.webkit.org/show_bug.cgi?id=49045 and is not related to the test failure in Qt.

[Csaba Osztrogonác]

svg/animations/animate-text-nested-transforms.html introduced in r71726
and has same failure as svg/animations/animate-path-nested-transforms.html
and was added to the skipped list: http://trac.webkit.org/changeset/71731

[Shane Stephens]

Sorry about this, I had intended to wait until the Qt issues were resolved before landing the second patch :(

I'll have access to a Linux box again in 2 days, if there's no progress by then I'll look into what's happening.

[Dirk Schulze]

*** This bug has been marked as a duplicate of bug 49045 ***