Bug 1245424 Part 2 - Stop checking the existence of details frame before toggling. r=bz

MozReview-Commit-ID: 6Xgyu7DDquy

--HG--
extra : rebase_source : 6b5ed8fa20117fcafea7dde779c550f6fd79be34

dom/html/HTMLSummaryElement.cpp

@@ -62,13 +62,11 @@ HTMLSummaryElement::PostHandleEvent(EventChainPostVisitor& aVisitor)
 
       // When dispatching a synthesized mouse click event to a details element
       // with 'display: none', both Chrome and Safari do not toggle the 'open'
-      // attribute. We follow them by checking whether the details element has a
-      // frame or not.
-      if (details->GetPrimaryFrame(Flush_Frames)) {
-        details->ToggleOpen();
-        aVisitor.mEventStatus = nsEventStatus_eConsumeNoDefault;
-        return NS_OK;
-      }
+      // attribute. We had tried to be compatible with this behavior, but found
+      // more inconsistency in test cases in bug 1245424. So we stop doing that.
+      details->ToggleOpen();
+      aVisitor.mEventStatus = nsEventStatus_eConsumeNoDefault;
+      return NS_OK;
     }
   } // event->HasMouseEventMessage()
 

layout/reftests/details-summary/reftest.list

@@ -65,7 +65,7 @@ pref(dom.details_element.enabled,true) == mouse-click-overflow-hidden-details.ht
 pref(dom.details_element.enabled,true) == mouse-click-twice-overflow-hidden-details.html overflow-hidden-details.html
 pref(dom.details_element.enabled,true) == mouse-click-overflow-auto-details.html overflow-auto-open-details.html
 pref(dom.details_element.enabled,true) == mouse-click-twice-overflow-auto-details.html overflow-auto-details.html
-pref(dom.details_element.enabled,true) == mouse-click-display-none-details.html single-summary.html
+pref(dom.details_element.enabled,true) == mouse-click-display-none-details.html open-single-summary.html
 
 # Dispatch mouse click to out-of-flow details or summary
 pref(dom.details_element.enabled,true) == mouse-click-fixed-summary.html open-fixed-summary.html