getElementsByTagName() should take a qualifiedName in parameter

third_party/WebKit/Source/core/dom/ContainerNode.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2013 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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "core/dom/ContainerNode.h"
 
 #include "bindings/core/v8/ExceptionState.h"
+#include "core/dom/AllDescendantsCollection.h"
 #include "core/dom/ChildFrameDisconnector.h"
 #include "core/dom/ChildListMutationScope.h"
 #include "core/dom/ClassCollection.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/NameNodeList.h"
 #include "core/dom/NodeChildRemovalTracker.h"
 #include "core/dom/NodeComputedStyle.h"
 #include "core/dom/NodeRareData.h"
 #include "core/dom/NodeTraversal.h"
@@ skipping 1396 matching lines @@
     return;
 
   GetDocument().InvalidateNodeListCaches(attr_name);
 
   for (ContainerNode* node = this; node; node = node->parentNode()) {
     if (NodeListsNodeData* lists = node->NodeLists())
       lists->InvalidateCaches(attr_name);
   }
 }
 
-TagCollection* ContainerNode::getElementsByTagName(
-    const AtomicString& local_name) {
-  if (GetDocument().IsHTMLDocument())
+HTMLCollection* ContainerNode::getElementsByTagName(
+    const AtomicString& qualified_name) {
+  DCHECK(!qualified_name.IsNull());
+
+  if (qualified_name == g_star_atom) {
+    return EnsureCachedCollection<AllDescendantsCollection>(
+        kAllDescendantsCollectionType);
+  }
+
+  if (GetDocument().IsHTMLDocument()) {
     return EnsureCachedCollection<HTMLTagCollection>(kHTMLTagCollectionType,
-                                                     local_name);
-  return EnsureCachedCollection<TagCollection>(kTagCollectionType, local_name);
+                                                     qualified_name);
+  }
+  return EnsureCachedCollection<TagCollection>(kTagCollectionType,
+                                               qualified_name);
 }
 
-TagCollection* ContainerNode::getElementsByTagNameNS(
+HTMLCollection* ContainerNode::getElementsByTagNameNS(
     const AtomicString& namespace_uri,
     const AtomicString& local_name) {
-  if (namespace_uri == g_star_atom)
-    return getElementsByTagName(local_name);
-
-  return EnsureCachedCollection<TagCollection>(
-      kTagCollectionType, namespace_uri.IsEmpty() ? g_null_atom : namespace_uri,
-      local_name);
+  return EnsureCachedCollection<TagCollectionNS>(
+      kTagCollectionNSType,
+      namespace_uri.IsEmpty() ? g_null_atom : namespace_uri, local_name);
 }
 
 // Takes an AtomicString in argument because it is common for elements to share
 // the same name attribute.  Therefore, the NameNodeList factory function
 // expects an AtomicString type.
 NameNodeList* ContainerNode::getElementsByName(
     const AtomicString& element_name) {
   return EnsureCachedCollection<NameNodeList>(kNameNodeListType, element_name);
 }
 
@@ skipping 50 matching lines @@
     return true;
 
   if (node->IsElementNode() && ToElement(node)->Shadow())
     return true;
 
   return false;
 }
 #endif
 
 }  // namespace blink