Support slotchange events

third_party/WebKit/Source/core/html/HTMLSlotElement.cpp

 /*
  * Copyright (C) 2015 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ 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 "core/html/HTMLSlotElement.h"
 
 #include "core/HTMLNames.h"
+#include "core/dom/Microtask.h"
 #include "core/dom/NodeTraversal.h"
 #include "core/dom/shadow/ElementShadow.h"
 #include "core/dom/shadow/InsertionPoint.h"
+#include "core/events/Event.h"
 #include "core/html/AssignedNodesOptions.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 inline HTMLSlotElement::HTMLSlotElement(Document& document)
     : HTMLElement(slotTag, document)
 {
 }
@@ skipping 49 matching lines @@
     m_distributedNodes.appendVector(other.m_distributedNodes);
     for (const auto& it : other.m_distributedIndices) {
         const Node* node = it.key;
         m_distributedIndices.set(node, index++);
     }
 }
 
 void HTMLSlotElement::clearDistribution()
 {
     m_assignedNodes.clear();
+    m_oldDistributedNodes.swap(m_distributedNodes);

[kochi, 2016/02/18 03:40:56]

Is there any possibility that clearDistribution() is called twice without
updateDistribution() in between?

[hayato, 2016/02/18 03:51:56]

It could happen in the future. I'm aware the current approach is very weak to
the future change.
See TODO comment here:
https://codereview.chromium.org/1695163003/diff/100001/third_party/WebKit/Sou...

I have a plan to use RAAI approach to get rid of m_oldDistributedNodes.

     m_distributedNodes.clear();
     m_distributedIndices.clear();
 }
 
+bool HTMLSlotElement::hasSlotChangeEventListener()
+{
+    return eventTargetData() && eventTargetData()->eventListenerMap.find(EventTypeNames::slotchange);

[esprehn, 2016/02/18 17:30:43]

this is not correct per the semantics of events, you should be able to do:

mutateDom();
Promise.resolve().then(() => {
  dom.addEventListener("slotchanged", () => {
    // this should get called.
  });
});

so you can't skip the microtask step to fire the event, the event must "always
happen", even if no one is listening.

this means events are a bad fit, it forces you to burn battery every time
there's a mutation even if no one is listening.

+}
+
+void HTMLSlotElement::dispatchSlotChangeEvent()
+{
+    RefPtrWillBeRawPtr<Event> event = Event::create(EventTypeNames::slotchange);
+    event->setTarget(this);
+    dispatchScopedEvent(event);
+}
+
 Node* HTMLSlotElement::distributedNodeNextTo(const Node& node) const
 {
     const auto& it = m_distributedIndices.find(&node);
     if (it == m_distributedIndices.end())
         return nullptr;
     size_t index = it->value;
     if (index + 1 == m_distributedNodes.size())
         return nullptr;
     return m_distributedNodes[index + 1].get();
 }
@@ skipping 91 matching lines @@
         // Insertion points are not supported as slots fallback
         if (isActiveInsertionPoint(child))
             continue;
         if (isHTMLSlotElement(child))
             appendDistributedNodesFrom(toHTMLSlotElement(child));
         else
             appendDistributedNode(child);
     }
 }
 
+void HTMLSlotElement::didUpdateDistribution()
+{
+    if (hasSlotChangeEventListener() && m_distributedNodes != m_oldDistributedNodes) {
+        // TODO(hayato): Do not enqueue a slotchange event for the same slot twice in the microtask queue
+        Microtask::enqueueMicrotask(WTF::bind(&HTMLSlotElement::dispatchSlotChangeEvent, PassRefPtrWillBeRawPtr<HTMLSlotElement>(this)));

[esprehn, 2016/02/18 17:30:43]

the de-duplication here is just reinventing what MutationObserver already does
:(

+    }
+    // TODO(hayato): Call setNeedsDistributionRecalc if the distribution changes due to the fallback elements
+}
+
 DEFINE_TRACE(HTMLSlotElement)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_assignedNodes);
     visitor->trace(m_distributedNodes);
     visitor->trace(m_distributedIndices);
+    visitor->trace(m_oldDistributedNodes);
 #endif
     HTMLElement::trace(visitor);
 }
 
 } // namespace blink