Revert of Implement script-side of callback reactions for Custom Elements V1

third_party/WebKit/Source/core/dom/custom/CustomElementDefinition.h

 // 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.
 
 #ifndef CustomElementDefinition_h
 #define CustomElementDefinition_h
 
 #include "bindings/core/v8/ScriptValue.h"
 #include "core/CoreExport.h"
 #include "core/dom/custom/CustomElementDescriptor.h"
 #include "platform/heap/Handle.h"
-#include "wtf/HashSet.h"
 #include "wtf/Noncopyable.h"
-#include "wtf/text/AtomicString.h"
-#include "wtf/text/AtomicStringHash.h"
 
 namespace blink {
 
 class Element;
 class ExceptionState;
 class HTMLElement;
 class QualifiedName;
 
 class CORE_EXPORT CustomElementDefinition
     : public GarbageCollectedFinalized<CustomElementDefinition> {
     WTF_MAKE_NONCOPYABLE(CustomElementDefinition);
 public:
     CustomElementDefinition(const CustomElementDescriptor&);
-    CustomElementDefinition(const CustomElementDescriptor&,
-        const HashSet<AtomicString>&);
     virtual ~CustomElementDefinition();
 
     DECLARE_VIRTUAL_TRACE();
 
     const CustomElementDescriptor& descriptor() { return m_descriptor; }
 
     // TODO(yosin): To support Web Modules, introduce an abstract
     // class |CustomElementConstructor| to allow us to have JavaScript
     // and C++ constructors and ask the binding layer to convert
     // |CustomElementConstructor| to |ScriptValue|. Replace
     // |getConstructorForScript()| by |getConstructor() ->
     // CustomElementConstructor|.
     virtual ScriptValue getConstructorForScript() = 0;
 
     using ConstructionStack = HeapVector<Member<Element>, 1>;
     ConstructionStack& constructionStack()
     {
         return m_constructionStack;
     }
 
     virtual HTMLElement* createElementSync(Document&, const QualifiedName&) = 0;
     virtual HTMLElement* createElementSync(Document&, const QualifiedName&, ExceptionState&) = 0;
     HTMLElement* createElementAsync(Document&, const QualifiedName&);
 
     void upgrade(Element*);
 
-    virtual bool hasConnectedCallback() const = 0;
-    virtual bool hasDisconnectedCallback() const = 0;
-    bool hasAttributeChangedCallback(const QualifiedName&);
+    // TODO(kojii): Change these methods to pure when script-side is implemented.
+    virtual bool hasConnectedCallback() const { return true; }
+    virtual bool hasDisconnectedCallback() const { return true; }
+    virtual bool hasAttributeChangedCallback(const QualifiedName&) const { return true; }
 
-    virtual void runConnectedCallback(Element*) = 0;
-    virtual void runDisconnectedCallback(Element*) = 0;
+    virtual void runConnectedCallback(Element*) {}
+    virtual void runDisconnectedCallback(Element*) {}
     virtual void runAttributeChangedCallback(Element*, const QualifiedName&,
-        const AtomicString& oldValue, const AtomicString& newValue) = 0;
+        const AtomicString& oldValue, const AtomicString& newValue) {}
 
     void enqueueUpgradeReaction(Element*);
     void enqueueConnectedCallback(Element*);
     void enqueueDisconnectedCallback(Element*);
     void enqueueAttributeChangedCallback(Element*, const QualifiedName&,
         const AtomicString& oldValue, const AtomicString& newValue);
 
 protected:
     virtual bool runConstructor(Element*) = 0;
 
     static void checkConstructorResult(Element*, Document&, const QualifiedName&, ExceptionState&);
 
-    HashSet<AtomicString> m_observedAttributes;
-
 private:
     const CustomElementDescriptor m_descriptor;
     ConstructionStack m_constructionStack;
-
-    void enqueueAttributeChangedCallbackForAllAttributes(Element*);
 };
 
 } // namespace blink
 
 #endif // CustomElementDefinition_h