Do refactor in collectGradientAttributes() and renderStyleForLengthResolve()

Source/core/svg/SVGRadialGradientElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 115 matching lines @@
 }
 
 RenderObject* SVGRadialGradientElement::createRenderer(RenderStyle*)
 {
     return new RenderSVGResourceRadialGradient(this);
 }
 
 bool SVGRadialGradientElement::collectGradientAttributes(RadialGradientAttributes& attributes)
 {
     HashSet<SVGGradientElement*> processedGradients;
-
-    bool isRadial = true;
     SVGGradientElement* current = this;
 
-    while (current) {
+    do {
         if (!current->renderer())
             return false;
 
         if (!attributes.hasSpreadMethod() && current->hasAttribute(SVGNames::spreadMethodAttr))
             attributes.setSpreadMethod(current->spreadMethodCurrentValue());
 
         if (!attributes.hasGradientUnits() && current->hasAttribute(SVGNames::gradientUnitsAttr))
             attributes.setGradientUnits(current->gradientUnitsCurrentValue());
 
         if (!attributes.hasGradientTransform() && current->hasAttribute(SVGNames::gradientTransformAttr)) {
             AffineTransform transform;
             current->gradientTransformCurrentValue().concatenate(transform);
             attributes.setGradientTransform(transform);
         }
 
         if (!attributes.hasStops()) {
             const Vector<Gradient::ColorStop>& stops(current->buildStops());
             if (!stops.isEmpty())
                 attributes.setStops(stops);
         }
 
-        if (isRadial) {
+        if (current->hasTagName(SVGNames::radialGradientTag)) {
             SVGRadialGradientElement* radial = toSVGRadialGradientElement(current);
 
             if (!attributes.hasCx() && current->hasAttribute(SVGNames::cxAttr))
                 attributes.setCx(radial->cx()->currentValue());
 
             if (!attributes.hasCy() && current->hasAttribute(SVGNames::cyAttr))
                 attributes.setCy(radial->cy()->currentValue());
 
             if (!attributes.hasR() && current->hasAttribute(SVGNames::rAttr))
                 attributes.setR(radial->r()->currentValue());
@@ skipping 13 matching lines @@
         // Respect xlink:href, take attributes from referenced element
         Node* refNode = SVGURIReference::targetElementFromIRIString(current->hrefCurrentValue(), document());
         if (refNode && (refNode->hasTagName(SVGNames::radialGradientTag) || refNode->hasTagName(SVGNames::linearGradientTag))) {
             current = toSVGGradientElement(refNode);
 
             // Cycle detection
             if (processedGradients.contains(current)) {
                 current = 0;
                 break;
             }
-
-            isRadial = current->hasTagName(SVGNames::radialGradientTag);
         } else
             current = 0;
-    }
+    } while (current);
 
     // Handle default values for fx/fy
     if (!attributes.hasFx())
         attributes.setFx(attributes.cx());
 
     if (!attributes.hasFy())
         attributes.setFy(attributes.cy());
     return true;
 }
 
 bool SVGRadialGradientElement::selfHasRelativeLengths() const
 {
     return m_cx->currentValue()->isRelative()
         || m_cy->currentValue()->isRelative()
         || m_r->currentValue()->isRelative()
         || m_fx->currentValue()->isRelative()
         || m_fy->currentValue()->isRelative()
         || m_fr->currentValue()->isRelative();
 }
 
 }