Have getElementsByClassName() / getElementsByTagName*() return an HTMLCollection

Source/core/fetch/FontResource.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile, Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 19 matching lines @@
 #include "core/fetch/ResourceClientWalker.h"
 #include "core/html/parser/TextResourceDecoder.h"
 #include "platform/SharedBuffer.h"
 #include "platform/fonts/FontCustomPlatformData.h"
 #include "platform/fonts/FontPlatformData.h"
 #include "public/platform/Platform.h"
 #include "wtf/CurrentTime.h"
 
 #if ENABLE(SVG_FONTS)
 #include "SVGNames.h"
-#include "core/dom/NodeList.h"
+#include "core/html/HTMLCollection.h"
 #include "core/svg/SVGDocument.h"
 #include "core/svg/SVGFontElement.h"
 #endif
 
 namespace WebCore {
 
 FontResource::FontResource(const ResourceRequest& resourceRequest)
     : Resource(resourceRequest, Font)
     , m_loadInitiated(false)
 {
@@ skipping 69 matching lines @@
         }
         if (!m_externalSVGDocument)
             setStatus(DecodeError);
     }
 
     return m_externalSVGDocument;
 }
 
 SVGFontElement* FontResource::getSVGFontById(const String& fontName) const
 {
-    RefPtr<NodeList> list = m_externalSVGDocument->getElementsByTagNameNS(SVGNames::fontTag.namespaceURI(), SVGNames::fontTag.localName());
-    if (!list)
+    RefPtr<HTMLCollection> collection = m_externalSVGDocument->getElementsByTagNameNS(SVGNames::fontTag.namespaceURI(), SVGNames::fontTag.localName());
+    if (!collection)
         return 0;
 
-    unsigned listLength = list->length();
-    if (!listLength)
+    unsigned collectionLength = collection->length();
+    if (!collectionLength)
         return 0;
 
 #ifndef NDEBUG
-    for (unsigned i = 0; i < listLength; ++i) {
-        ASSERT(list->item(i));
-        ASSERT(list->item(i)->hasTagName(SVGNames::fontTag));
+    for (unsigned i = 0; i < collectionLength; ++i) {
+        ASSERT(collection->item(i));
+        ASSERT(collection->item(i)->hasTagName(SVGNames::fontTag));
     }
 #endif
 
     if (fontName.isEmpty())
-        return toSVGFontElement(list->item(0));
+        return toSVGFontElement(collection->item(0));
 
-    for (unsigned i = 0; i < listLength; ++i) {
-        SVGFontElement* element = toSVGFontElement(list->item(i));
+    for (unsigned i = 0; i < collectionLength; ++i) {
+        SVGFontElement* element = toSVGFontElement(collection->item(i));
         if (element->getIdAttribute() == fontName)
             return element;
     }
 
     return 0;
 }
 #endif
 
 void FontResource::allClientsRemoved()
 {
     m_fontData.clear();
     Resource::allClientsRemoved();
 }
 
 void FontResource::checkNotify()
 {
     ResourceClientWalker<FontResourceClient> w(m_clients);
     while (FontResourceClient* c = w.next())
         c->fontLoaded(this);
 }
 
 }