Implement Custom Elements' entered and left document callbacks.

Source/core/dom/CustomElementCallbackInvocation.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * 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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/dom/CustomElementCallbackInvocation.h"
 
-#include "core/dom/CustomElementLifecycleCallbacks.h"
+#include "core/dom/CustomElementCallbackDispatcher.h"
 
 namespace WebCore {
 
 class AttributeChangedInvocation : public CustomElementCallbackInvocation {

[Yuta Kitamura, 2013/07/08 03:40:49]

The separation between AttributeChangedInvocation and CECallbackInvocation
looks unclear to me. AttributeChangedInvocation is special-cased
just bacause it has extra parameters to save; am I correct?

If so, the class hierarchy looks a bit awkward to me. I think we should have
a pure interface that just defines a pure virtual function dispatch() and
both AttributeChangedInvocation and CECallbackInvocation can be derived from it.

 public:
-    AttributeChangedInvocation(const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue);
+    AttributeChangedInvocation(PassRefPtr<CustomElementLifecycleCallbacks>, const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue);
 
 private:
-    virtual void dispatch(CustomElementLifecycleCallbacks*, Element*) OVERRIDE;
+    virtual void dispatch(Element*) OVERRIDE;
 
     AtomicString m_name;
     AtomicString m_oldValue;
     AtomicString m_newValue;
 };
 
-class CreatedInvocation : public CustomElementCallbackInvocation {
-public:
-    CreatedInvocation() { }
-private:
-    virtual void dispatch(CustomElementLifecycleCallbacks*, Element*) OVERRIDE;
-};
+PassOwnPtr<CustomElementCallbackInvocation> CustomElementCallbackInvocation::createInvocation(PassRefPtr<CustomElementLifecycleCallbacks> callbacks, CustomElementLifecycleCallbacks::CallbackType which)
+{
+    switch (which) {
+    case CustomElementLifecycleCallbacks::Created:
+    case CustomElementLifecycleCallbacks::EnteredDocument:
+    case CustomElementLifecycleCallbacks::LeftDocument:

[Yuta Kitamura, 2013/07/08 03:40:49]

CELifecycleCallbacks::CallbackType looks like a bitmask. Is it possible
that these values are combined (e.g. "Created | EnteredDocument")?
If not, why don't you ASSERT that in the constructor?

+        return adoptPtr(new CustomElementCallbackInvocation(callbacks, which));
 
-PassOwnPtr<CustomElementCallbackInvocation> CustomElementCallbackInvocation::createCreatedInvocation()
-{
-    return adoptPtr(new CreatedInvocation());
+    case CustomElementLifecycleCallbacks::AttributeChanged: // Use createAttributeChangedInvocation
+    default:
+        ASSERT_NOT_REACHED();
+        return PassOwnPtr<CustomElementCallbackInvocation>();
+    }
 }
 
-PassOwnPtr<CustomElementCallbackInvocation> CustomElementCallbackInvocation::createAttributeChangedInvocation(const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue)
+void CustomElementCallbackInvocation::dispatch(Element* element)
 {
-    return adoptPtr(new AttributeChangedInvocation(name, oldValue, newValue));
+    switch (m_which) {
+    case CustomElementLifecycleCallbacks::Created:
+        if (element->inDocument())
+            CustomElementCallbackDispatcher::instance().enqueueEnteredDocumentCallback(m_callbacks, element);
+        m_callbacks->created(element);
+        break;
+    case CustomElementLifecycleCallbacks::EnteredDocument:
+        m_callbacks->enteredDocument(element);
+        break;
+    case CustomElementLifecycleCallbacks::LeftDocument:
+        m_callbacks->leftDocument(element);
+        break;
+    default:
+        ASSERT_NOT_REACHED();
+    }
 }
 
-AttributeChangedInvocation::AttributeChangedInvocation(const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue)
-    : m_name(name)
+PassOwnPtr<CustomElementCallbackInvocation> CustomElementCallbackInvocation::createAttributeChangedInvocation(PassRefPtr<CustomElementLifecycleCallbacks> callbacks, const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue)
+{
+    return adoptPtr(new AttributeChangedInvocation(callbacks, name, oldValue, newValue));
+}
+
+AttributeChangedInvocation::AttributeChangedInvocation(PassRefPtr<CustomElementLifecycleCallbacks> callbacks, const AtomicString& name, const AtomicString& oldValue, const AtomicString& newValue)
+    : CustomElementCallbackInvocation(callbacks, CustomElementLifecycleCallbacks::AttributeChanged)
+    , m_name(name)
     , m_oldValue(oldValue)
     , m_newValue(newValue)
 {
 }
 
-void AttributeChangedInvocation::dispatch(CustomElementLifecycleCallbacks* callbacks, Element* element)
+void AttributeChangedInvocation::dispatch(Element* element)
 {
-    callbacks->attributeChanged(element, m_name, m_oldValue, m_newValue);
-}
-
-void CreatedInvocation::dispatch(CustomElementLifecycleCallbacks* callbacks, Element* element)
-{
-    callbacks->created(element);
+    callbacks()->attributeChanged(element, m_name, m_oldValue, m_newValue);
 }
 
 } // namespace WebCore