HTMLDetailsElement should fire a 'toggle' event

Source/core/html/HTMLDetailsElement.cpp

 /*
  * Copyright (C) 2010, 2011 Nokia Corporation and/or its subsidiary(-ies)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 29 matching lines @@
 
 PassRefPtr<HTMLDetailsElement> HTMLDetailsElement::create(Document& document)
 {
     RefPtr<HTMLDetailsElement> details = adoptRef(new HTMLDetailsElement(document));
     details->ensureUserAgentShadowRoot();
     return details.release();
 }
 
 HTMLDetailsElement::HTMLDetailsElement(Document& document)
     : HTMLElement(detailsTag, document)
+    , m_toggleEventTimer(this, &HTMLDetailsElement::toggleEventTimerFired)
     , m_isOpen(false)
 {
     ScriptWrappable::init(this);
 }
 
+void HTMLDetailsElement::toggleEventTimerFired(Timer<HTMLDetailsElement> *)
+{
+    dispatchEvent(Event::create(EventTypeNames::toggle));
+}
+
+void HTMLDetailsElement::dispatchToggleEventAsync()
+{
+    if (m_toggleEventTimer.isActive())
+        m_toggleEventTimer.stop();
+    m_toggleEventTimer.startOneShot(0, FROM_HERE);
+}
+
 RenderObject* HTMLDetailsElement::createRenderer(RenderStyle*)
 {
     return new RenderBlockFlow(this);
 }
 
 void HTMLDetailsElement::didAddUserAgentShadowRoot(ShadowRoot& root)
 {
     DEFINE_STATIC_LOCAL(const AtomicString, summarySelector, ("summary:first-of-type", AtomicString::ConstructFromLiteral));
 
     RefPtr<HTMLSummaryElement> defaultSummary = HTMLSummaryElement::create(document());
@@ skipping 22 matching lines @@
     return toElement(content->firstChild());
 }
 
 void HTMLDetailsElement::parseAttribute(const QualifiedName& name, const AtomicString& value)
 {
     if (name == openAttr) {
         bool oldValue = m_isOpen;
         m_isOpen = !value.isNull();
         if (m_isOpen == oldValue)
             return;
+
+        // Dispatch toggle event asynchronously.
+        dispatchToggleEventAsync();
+
         Element* content = ensureUserAgentShadowRoot().getElementById(ShadowElementNames::detailsContent());
         ASSERT(content);
         if (m_isOpen)
             content->removeInlineStyleProperty(CSSPropertyDisplay);
         else
             content->setInlineStyleProperty(CSSPropertyDisplay, CSSValueNone);
         Element* summary = ensureUserAgentShadowRoot().getElementById(ShadowElementNames::detailsSummary());
         ASSERT(summary);
         // FIXME: DetailsMarkerControl's RenderDetailsMarker has no concept of being updated
         // without recreating it causing a repaint. Instead we should change it so we can tell
         // it to toggle the open/closed triangle state and avoid reattaching the entire summary.
         summary->lazyReattachIfAttached();
         return;
     }
     HTMLElement::parseAttribute(name, value);
 }
 
 void HTMLDetailsElement::toggleOpen()
 {
     setAttribute(openAttr, m_isOpen ? nullAtom : emptyAtom);
 }
 
 bool HTMLDetailsElement::isInteractiveContent() const
 {
     return true;
 }
 
 }