Improve a search logic to find a SVGFontFaceSrcElement

Source/core/svg/SVGFontFaceElement.cpp

 /*
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2008 Apple Inc. 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
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 255 matching lines @@
     return m_fontElement;
 }
 
 void SVGFontFaceElement::rebuildFontFace()
 {
     if (!inDocument()) {
         ASSERT(!m_fontElement);
         return;
     }
 
-    // we currently ignore all but the first src element, alternatively we could concat them
-    SVGFontFaceSrcElement* srcElement = 0;
-
-    for (Node* child = firstChild(); child && !srcElement; child = child->nextSibling()) {
-        if (child->hasTagName(font_face_srcTag))
-            srcElement = static_cast<SVGFontFaceSrcElement*>(child);
-    }
-
     bool describesParentFont = parentNode()->hasTagName(SVGNames::fontTag);
     RefPtr<CSSValueList> list;
 
     if (describesParentFont) {
         m_fontElement = toSVGFontElement(parentNode());
 
         list = CSSValueList::createCommaSeparated();
         list->append(CSSFontFaceSrcValue::createLocal(fontFamily()));
     } else {
         m_fontElement = 0;
-        if (srcElement)
-            list = srcElement->srcValue();
+        // we currently ignore all but the last src element, alternatively we could concat them
+        for (Node* child = lastChild(); child && !list; child = child->previousSibling()) {
+            if (child->hasTagName(font_face_srcTag)) {
+                list = static_cast<SVGFontFaceSrcElement*>(child)->srcValue();
+                break;
+            }
+        }
     }
 
     if (!list || !list->length())
         return;
 
     // Parse in-memory CSS rules
     m_fontFaceRule->mutableProperties()->addParsedProperty(CSSProperty(CSSPropertySrc, list));
 
     if (describesParentFont) {
         // Traverse parsed CSS values and associate CSSFontFaceSrcValue elements with ourselves.
@@ skipping 39 matching lines @@
 
 void SVGFontFaceElement::childrenChanged(bool changedByParser, Node* beforeChange, Node* afterChange, int childCountDelta)
 {
     SVGElement::childrenChanged(changedByParser, beforeChange, afterChange, childCountDelta);
     rebuildFontFace();
 }
 
 } // namespace WebCore
 
 #endif // ENABLE(SVG_FONTS)