Fire attributeChangedCallback on style changes for Custom Elements V1

third_party/WebKit/Source/core/dom/custom/CustomElementsRegistryTest.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/CustomElementsRegistry.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptValue.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
@@ skipping 213 matching lines @@
 }
 
 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()
@@ skipping 14 matching lines @@
         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)
+        : TestCustomElementDefinition(descriptor, {
+            "attr1",
+            "attr2",
+            HTMLNames::contenteditableAttr.localName(),
+        })
     {
-        m_observedAttributes.add("attr1");
-        m_observedAttributes.add("attr2");
-        m_observedAttributes.add(HTMLNames::contenteditableAttr.localName());
     }
 
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
         TestCustomElementDefinition::trace(visitor);
         visitor->trace(m_element);
     }
 
     // TODO(dominicc): Make this class collect a vector of what's
     // upgraded; it will be useful in more tests.
@@ skipping 178 matching lines @@
         << "remove() should invoke disconnectedCallback";
 
     EXPECT_EQ(1u, definition->m_logs.size())
         << "remove() should not invoke other callbacks";
 }
 
 // TODO(dominicc): Add tests which adjust the "is" attribute when type
 // extensions are implemented.
 
 } // namespace blink