Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp

Issue 2549973003: Revert commit# 415577 "Add grid/flex layout support for <fieldset>" (Closed)
Patch Set: updated TestExpectations, verified that there are no changes in images Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
index 10507de24c542e11585ac683879858401dc1b4d7..fc80cc42c085f3105dc91a4ed15fe70f42bccf71 100644
--- a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
@@ -937,11 +937,6 @@ void LayoutFlexibleBox::layoutFlexItems(bool relayoutChildren,
PaintLayerScrollableArea::PreventRelayoutScope preventRelayoutScope(
layoutScope);
- // Fieldsets need to find their legend and position it inside the border of
- // the object. The legend then gets skipped during normal layout. It
- // doesn't get included in the normal layout process but is instead skipped.
- LayoutObject* childToExclude =
- layoutSpecialExcludedChild(relayoutChildren, layoutScope);
// Set up our master list of flex items. All of the rest of the algorithm
// should work off this list of a subset.
@@ -951,9 +946,6 @@ void LayoutFlexibleBox::layoutFlexItems(bool relayoutChildren,
m_orderIterator.first();
for (LayoutBox* child = m_orderIterator.currentChild(); child;
child = m_orderIterator.next()) {
- if (childToExclude == child)
- continue; // Skip this child, since it will be positioned by the
- // specialized subclass (fieldsets runs).
if (child->isOutOfFlowPositioned()) {
// Out-of-flow children are not flex items, so we skip them here.

Powered by Google App Engine
This is Rietveld 408576698