Do refactor in collectGradientAttributes() and renderStyleForLengthResolve()

Source/core/svg/SVGLinearGradientElement.cpp

Index: Source/core/svg/SVGLinearGradientElement.cpp
diff --git a/Source/core/svg/SVGLinearGradientElement.cpp b/Source/core/svg/SVGLinearGradientElement.cpp
index 19df4d50b46b7eedc0f0ac5a5057ca7f93c6c38b..72e584755ccd19f2aff7be8627c2eb3a868d0082 100644
--- a/Source/core/svg/SVGLinearGradientElement.cpp
+++ b/Source/core/svg/SVGLinearGradientElement.cpp
@@ -119,10 +119,9 @@ bool SVGLinearGradientElement::collectGradientAttributes(LinearGradientAttribute
 {
     HashSet<SVGGradientElement*> processedGradients;
 
-    bool isLinear = true;
     SVGGradientElement* current = this;
 
-    while (current) {
+    do {
         if (!current->renderer())
             return false;
 
@@ -144,7 +143,7 @@ bool SVGLinearGradientElement::collectGradientAttributes(LinearGradientAttribute
                 attributes.setStops(stops);
         }
 
-        if (isLinear) {
+        if (current->hasTagName(SVGNames::linearGradientTag)) {
             SVGLinearGradientElement* linear = toSVGLinearGradientElement(current);
 
             if (!attributes.hasX1() && current->hasAttribute(SVGNames::x1Attr))
@@ -172,11 +171,9 @@ bool SVGLinearGradientElement::collectGradientAttributes(LinearGradientAttribute
                 current = 0;
                 break;
             }
-
-            isLinear = current->hasTagName(SVGNames::linearGradientTag);
         } else
             current = 0;

[Stephen Chennney, 2014/01/14 13:44:16]

Need braces here.

-    }
+    } while (current);
 
     return true;
 }