Make createElement tag name case handling consistent for custom elements

third_party/WebKit/Source/core/dom/custom/CustomElementRegistryTest.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/dom/custom/CustomElementRegistry.h"
 
 #include "bindings/core/v8/ExceptionState.h"
-#include "bindings/core/v8/ScriptValue.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
 #include "core/dom/ElementDefinitionOptions.h"
 #include "core/dom/custom/CEReactionsScope.h"
 #include "core/dom/custom/CustomElementDefinition.h"
 #include "core/dom/custom/CustomElementDefinitionBuilder.h"
 #include "core/dom/custom/CustomElementDescriptor.h"
 #include "core/dom/custom/CustomElementTestHelpers.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/dom/shadow/ShadowRootInit.h"
@@ skipping 135 matching lines @@
   document().documentElement()->appendChild(elementC);
 
   HeapVector<Member<Element>> elements;
   collectCandidates(CustomElementDescriptor("a-a", "a-a"), &elements);
 
   EXPECT_EQ(elementA, elements[0].get());
   EXPECT_EQ(elementB, elements[1].get());
   EXPECT_EQ(elementC, elements[2].get());
 }
 
-class TestCustomElementDefinition : public CustomElementDefinition {
-  WTF_MAKE_NONCOPYABLE(TestCustomElementDefinition);
-
- public:
-  TestCustomElementDefinition(const CustomElementDescriptor& descriptor)
-      : CustomElementDefinition(descriptor) {}
-
-  TestCustomElementDefinition(const CustomElementDescriptor& descriptor,
-                              const HashSet<AtomicString>& observedAttributes)
-      : CustomElementDefinition(descriptor, observedAttributes) {}
-
-  ~TestCustomElementDefinition() override = default;
-
-  ScriptValue getConstructorForScript() override { return ScriptValue(); }
-
-  bool runConstructor(Element* element) override {
-    if (constructionStack().isEmpty() || constructionStack().last() != element)
-      return false;
-    constructionStack().last().clear();
-    return true;
-  }
-
-  HTMLElement* createElementSync(Document&, const QualifiedName&) override {
-    return nullptr;
-  }
-
-  HTMLElement* createElementSync(Document&,
-                                 const QualifiedName&,
-                                 ExceptionState&) override {
-    return nullptr;
-  }
-};
-
 // Classes which use trace macros cannot be local because of the
 // traceImpl template.
 class LogUpgradeDefinition : public TestCustomElementDefinition {
   WTF_MAKE_NONCOPYABLE(LogUpgradeDefinition);
 
  public:
   LogUpgradeDefinition(const CustomElementDescriptor& descriptor)
       : TestCustomElementDefinition(
             descriptor,
             {
@@ skipping 75 matching lines @@
   void runAttributeChangedCallback(Element* element,
                                    const QualifiedName& name,
                                    const AtomicString& oldValue,
                                    const AtomicString& newValue) override {
     m_logs.append(AttributeChangedCallback);
     EXPECT_EQ(element, m_element);
     m_attributeChanged.append(AttributeChanged{name, oldValue, newValue});
   }
 };
 
-class LogUpgradeBuilder final : public CustomElementDefinitionBuilder {
+class LogUpgradeBuilder final : public TestCustomElementDefinitionBuilder {
   STACK_ALLOCATED();
   WTF_MAKE_NONCOPYABLE(LogUpgradeBuilder);
 
  public:
   LogUpgradeBuilder() {}
 
-  bool checkConstructorIntrinsics() override { return true; }
-  bool checkConstructorNotRegistered() override { return true; }
-  bool checkPrototype() override { return true; }
-  bool rememberOriginalProperties() override { return true; }
-  CustomElementDefinition* build(const CustomElementDescriptor& descriptor) {
+  CustomElementDefinition* build(
+      const CustomElementDescriptor& descriptor) override {
     return new LogUpgradeDefinition(descriptor);
   }
 };
 
 TEST_F(CustomElementRegistryTest, define_upgradesInDocumentElements) {
   ScriptForbiddenScope doNotRelyOnScript;
 
   Element* element = CreateElement("a-a").inDocument(&document());
   element->setAttribute(
       QualifiedName(nullAtom, "attr1", HTMLNames::xhtmlNamespaceURI), "v1");
@@ skipping 134 matching lines @@
       << "adoptedCallback should have been passed the new owner document";
 
   EXPECT_EQ(2u, definition->m_logs.size())
       << "adoptNode() should not invoke other callbacks";
 }
 
 // TODO(dominicc): Add tests which adjust the "is" attribute when type
 // extensions are implemented.
 
 }  // namespace blink