2011-06-29 Ryan Sleevi <rsleevi@chromium.org>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Jun 2011 09:32:21 +0000 (09:32 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Jun 2011 09:32:21 +0000 (09:32 +0000)
        Reviewed by Dirk Schulze.

        SVGRadialGradientElement::selfHasRelativeLengths() doesn't consider if cx() is relative
        https://bugs.webkit.org/show_bug.cgi?id=61600

        Correct the logic for checking for relative lengths in a SVGRadialGradientElement to consider cx() in addition to cy()

        No new tests, as there is no way to currently test this. With the bug present, a side-effect is causing the
        expected/correct image to be rendered. This is because the cached SVG resource is getting invalided through a
        different path, forcing a repaint. The repaint takes into consideration the proper dimensions, hiding the bug.

        * svg/SVGRadialGradientElement.cpp:
        (WebCore::SVGRadialGradientElement::selfHasRelativeLengths):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@90003 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/svg/SVGRadialGradientElement.cpp

index 7b476c3..1856112 100644 (file)
@@ -1,3 +1,19 @@
+2011-06-29  Ryan Sleevi  <rsleevi@chromium.org>
+
+        Reviewed by Dirk Schulze.
+
+        SVGRadialGradientElement::selfHasRelativeLengths() doesn't consider if cx() is relative
+        https://bugs.webkit.org/show_bug.cgi?id=61600
+
+        Correct the logic for checking for relative lengths in a SVGRadialGradientElement to consider cx() in addition to cy()
+        
+        No new tests, as there is no way to currently test this. With the bug present, a side-effect is causing the
+        expected/correct image to be rendered. This is because the cached SVG resource is getting invalided through a
+        different path, forcing a repaint. The repaint takes into consideration the proper dimensions, hiding the bug.
+
+        * svg/SVGRadialGradientElement.cpp:
+        (WebCore::SVGRadialGradientElement::selfHasRelativeLengths):
+
 2011-06-28  Dirk Schulze  <krit@webkit.org>
 
         Reviewed by Nikolas Zimmermann.
index d75dcfc..88f1498 100644 (file)
@@ -305,7 +305,7 @@ void SVGRadialGradientElement::calculateFocalCenterPointsAndRadius(const RadialG
 
 bool SVGRadialGradientElement::selfHasRelativeLengths() const
 {
-    return cy().isRelative()
+    return cx().isRelative()
         || cy().isRelative()
         || r().isRelative()
         || fx().isRelative()