Bug 48987 - [Qt] svg/animations/animate-path-nested-transforms.html fails
Summary: [Qt] svg/animations/animate-path-nested-transforms.html fails
Status: RESOLVED DUPLICATE of bug 49045
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2010-11-04 04:05 PDT by Csaba Osztrogonác
Modified: 2011-01-23 14:29 PST (History)
7 users (show)

See Also:


Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2010-11-04 04:05:15 PDT
svg/animations/animate-path-nested-transforms.html introduced in r71314 and fails:
Comment 1 Csaba Osztrogonác 2010-11-04 04:19:18 PDT
I added svg/animations/animate-path-nested-transforms.html to
the Skipped list: http://trac.webkit.org/changeset/71320
Comment 2 Shane Stephens 2010-11-04 14:26:27 PDT
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.
Comment 3 Tony Chang 2010-11-04 14:33:32 PDT
Adding some people who might know about svg in qt.
Comment 4 Philippe Normand 2010-11-05 04:42:26 PDT
New test crashes on GTK, see bug 49065
Comment 5 Shane Stephens 2010-11-05 10:01:40 PDT
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.
Comment 6 Csaba Osztrogonác 2010-11-10 03:04:36 PST
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
Comment 7 Shane Stephens 2010-11-10 08:18:19 PST
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.
Comment 8 Dirk Schulze 2011-01-23 14:29:23 PST

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