Refactor calculation of hasRx and hasRy values in SVGPathData

https://bugs.webkit.org/show_bug.cgi?id=127423

Patch by Peter Molnar <pmolnar.u-szeged@partner.samsung.com> on 2014-01-22
Reviewed by Darin Adler.

This is a follow-up on https://bugs.webkit.org/show_bug.cgi?id=127337

* rendering/svg/SVGPathData.cpp:
(WebCore::updatePathFromRectElement):

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

Source/WebCore/ChangeLog

+2014-01-22  Peter Molnar  <pmolnar.u-szeged@partner.samsung.com>
+
+        Refactor calculation of hasRx and hasRy values in SVGPathData
+        https://bugs.webkit.org/show_bug.cgi?id=127423
+
+        Reviewed by Darin Adler.
+
+        This is a follow-up on https://bugs.webkit.org/show_bug.cgi?id=127337
+
+        * rendering/svg/SVGPathData.cpp:
+        (WebCore::updatePathFromRectElement):
+
 2014-01-21  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Remove recompileAllJSFunctions timer in ScriptDebugServer

Source/WebCore/rendering/svg/SVGPathData.cpp

@@ -115,11 +115,11 @@ static void updatePathFromRectElement(SVGElement* element, Path& path)
         return;
     float x = rect->x().value(lengthContext);
     float y = rect->y().value(lengthContext);
-    bool hasRx = rect->rx().value(lengthContext) > 0;
-    bool hasRy = rect->ry().value(lengthContext) > 0;
+    float rx = rect->rx().value(lengthContext);
+    float ry = rect->ry().value(lengthContext);
+    bool hasRx = rx > 0;
+    bool hasRy = ry > 0;
     if (hasRx || hasRy) {
-        float rx = rect->rx().value(lengthContext);
-        float ry = rect->ry().value(lengthContext);
         if (!hasRx)
             rx = ry;
         else if (!hasRy)