HTML - details/summary - lists (the "Second item" will be displayed as "3. Second item") - potential crashtests

This commit is contained in:
janekptacijarabaci
2017-11-28 10:00:31 +01:00
committed by Roy Tam
parent ce87aaec5b
commit 8bba1c2e08
12 changed files with 183 additions and 17 deletions
+5 -7
View File
@@ -65,13 +65,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()