Support display:flex for 'summary'

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

 /*
  * Copyright (C) 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 12 matching lines @@
 #include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/HTMLNames.h"
 #include "core/dom/shadow/FlatTreeTraversal.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/events/KeyboardEvent.h"
 #include "core/html/HTMLContentElement.h"
 #include "core/html/HTMLDetailsElement.h"
 #include "core/html/shadow/DetailsMarkerControl.h"
 #include "core/html/shadow/ShadowElementNames.h"
 #include "core/layout/LayoutBlockFlow.h"
+#include "core/layout/LayoutFlexibleBox.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 HTMLSummaryElement* HTMLSummaryElement::create(Document& document)
 {
     HTMLSummaryElement* summary = new HTMLSummaryElement(document);
     summary->ensureUserAgentShadowRoot();
     return summary;
 }
 
 HTMLSummaryElement::HTMLSummaryElement(Document& document)
     : HTMLElement(summaryTag, document)
 {
 }
 
-LayoutObject* HTMLSummaryElement::createLayoutObject(const ComputedStyle&)
+LayoutObject* HTMLSummaryElement::createLayoutObject(const ComputedStyle& style)
 {
+    if (style.display() == FLEX)

[cbiesinger, 2016/09/28 11:16:09]

Could you also add GRID and the inline variants here? (INLINE_FLEX, INLINE_GRID)

+        return LayoutObject::createObject(this, style);
     return new LayoutBlockFlow(this);
 }
 
 void HTMLSummaryElement::didAddUserAgentShadowRoot(ShadowRoot& root)
 {
     DetailsMarkerControl* markerControl = DetailsMarkerControl::create(document());
     markerControl->setIdAttribute(ShadowElementNames::detailsMarker());
     root.appendChild(markerControl);
     root.appendChild(HTMLContentElement::create(document()));
 }
@@ skipping 80 matching lines @@
 
 bool HTMLSummaryElement::willRespondToMouseClickEvents()
 {
     if (isMainSummary() && layoutObject())
         return true;
 
     return HTMLElement::willRespondToMouseClickEvents();
 }
 
 } // namespace blink