[webkit-changes] [111596] trunk/Source/WebCore

Wed, 21 Mar 2012 13:00:52 -0700

Title: [111596] trunk/Source/WebCore
Revision
111596
Author
[email protected]
Date
2012-03-21 13:00:22 -0700 (Wed, 21 Mar 2012)

Log Message

compute the sign of flexibility for new flexbox
https://bugs.webkit.org/show_bug.cgi?id=81722

Reviewed by Ojan Vafai.

This implements the computation of the sign of flexibility used by the
new flexing algorithm.
http://dev.w3.org/csswg/css3-flexbox/#resolve-the-flexible-lengths

No new tests, we don't use the sign of flexibility yet.

* rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::layoutFlexItems):
(WebCore::RenderFlexibleBox::computeNextFlexLine): Compute the size
adjusted for min/max as well as not adjusted.
(WebCore::RenderFlexibleBox::resolveFlexibleLengths): Renamed to match
spec text better.
* rendering/RenderFlexibleBox.h: Added enum for flex sign.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (111595 => 111596)


--- trunk/Source/WebCore/ChangeLog	2012-03-21 19:42:51 UTC (rev 111595)
+++ trunk/Source/WebCore/ChangeLog	2012-03-21 20:00:22 UTC (rev 111596)
@@ -1,3 +1,24 @@
+2012-03-21  Tony Chang  <[email protected]>
+
+        compute the sign of flexibility for new flexbox
+        https://bugs.webkit.org/show_bug.cgi?id=81722
+
+        Reviewed by Ojan Vafai.
+
+        This implements the computation of the sign of flexibility used by the
+        new flexing algorithm.
+        http://dev.w3.org/csswg/css3-flexbox/#resolve-the-flexible-lengths
+
+        No new tests, we don't use the sign of flexibility yet.
+
+        * rendering/RenderFlexibleBox.cpp:
+        (WebCore::RenderFlexibleBox::layoutFlexItems):
+        (WebCore::RenderFlexibleBox::computeNextFlexLine): Compute the size
+        adjusted for min/max as well as not adjusted.
+        (WebCore::RenderFlexibleBox::resolveFlexibleLengths): Renamed to match
+        spec text better.
+        * rendering/RenderFlexibleBox.h: Added enum for flex sign.
+
 2012-03-21  Vsevolod Vlasov  <[email protected]>
 
         Web Inspector: Extract WebInspector.UIBreakpoint from WebInspector.Breakpoint.

Modified: trunk/Source/WebCore/rendering/RenderFlexibleBox.cpp (111595 => 111596)


--- trunk/Source/WebCore/rendering/RenderFlexibleBox.cpp	2012-03-21 19:42:51 UTC (rev 111595)
+++ trunk/Source/WebCore/rendering/RenderFlexibleBox.cpp	2012-03-21 20:00:22 UTC (rev 111596)
@@ -606,6 +606,7 @@
     LayoutUnit preferredMainAxisExtent;
     float totalPositiveFlexibility;
     float totalNegativeFlexibility;
+    LayoutUnit minMaxAppliedMainAxisExtent;
     FlexOrderIterator flexIterator(this, flexOrderValues);
 
     // For wrap-reverse, we need to layout as wrap, then reverse the lines. The next two arrays
@@ -613,11 +614,12 @@
     WrapReverseContext wrapReverseContext(style()->flexWrap());
 
     LayoutUnit crossAxisOffset = flowAwareBorderBefore() + flowAwarePaddingBefore();
-    while (computeNextFlexLine(flexIterator, orderedChildren, preferredMainAxisExtent, totalPositiveFlexibility, totalNegativeFlexibility)) {
+    while (computeNextFlexLine(flexIterator, orderedChildren, preferredMainAxisExtent, totalPositiveFlexibility, totalNegativeFlexibility, minMaxAppliedMainAxisExtent)) {
         LayoutUnit availableFreeSpace = computeAvailableFreeSpace(preferredMainAxisExtent);
+        FlexSign flexSign = (minMaxAppliedMainAxisExtent < preferredMainAxisExtent + availableFreeSpace) ? PositiveFlexibility : NegativeFlexibility;
         InflexibleFlexItemSize inflexibleItems;
         WTF::Vector<LayoutUnit> childSizes;
-        while (!runFreeSpaceAllocationAlgorithm(orderedChildren, availableFreeSpace, totalPositiveFlexibility, totalNegativeFlexibility, inflexibleItems, childSizes)) {
+        while (!resolveFlexibleLengths(flexSign, orderedChildren, availableFreeSpace, totalPositiveFlexibility, totalNegativeFlexibility, inflexibleItems, childSizes)) {
             ASSERT(totalPositiveFlexibility >= 0 && totalNegativeFlexibility >= 0);
             ASSERT(inflexibleItems.size() > 0);
         }
@@ -703,15 +705,17 @@
     return height;
 }
 
-bool RenderFlexibleBox::computeNextFlexLine(FlexOrderIterator& iterator, OrderedFlexItemList& orderedChildren, LayoutUnit& preferredMainAxisExtent, float& totalPositiveFlexibility, float& totalNegativeFlexibility)
+bool RenderFlexibleBox::computeNextFlexLine(FlexOrderIterator& iterator, OrderedFlexItemList& orderedChildren, LayoutUnit& preferredMainAxisExtent, float& totalPositiveFlexibility, float& totalNegativeFlexibility, LayoutUnit& minMaxAppliedMainAxisExtent)
 {
     orderedChildren.clear();
     preferredMainAxisExtent = 0;
     totalPositiveFlexibility = totalNegativeFlexibility = 0;
+    minMaxAppliedMainAxisExtent = 0;
 
     if (!iterator.currentChild())
         return false;
 
+    LayoutUnit flexboxAvailableContentExtent = mainAxisContentExtent();
     LayoutUnit lineBreak = lineBreakLength();
 
     for (RenderBox* child = iterator.currentChild(); child; child = iterator.next()) {
@@ -720,24 +724,33 @@
             continue;
         }
 
-        LayoutUnit childMainAxisExtent = mainAxisBorderAndPaddingExtentForChild(child) + preferredMainAxisContentExtentForChild(child);
-        if (isHorizontalFlow())
-            childMainAxisExtent += child->marginWidth();
-        else
-            childMainAxisExtent += child->marginHeight();
+        LayoutUnit childMainAxisExtent = preferredMainAxisContentExtentForChild(child);
+        LayoutUnit childMainAxisMarginBoxExtent = mainAxisBorderAndPaddingExtentForChild(child) + childMainAxisExtent;
+        childMainAxisMarginBoxExtent += isHorizontalFlow() ? child->marginWidth() : child->marginHeight();
 
-        if (isMultiline() && preferredMainAxisExtent + childMainAxisExtent > lineBreak && orderedChildren.size() > 0)
+        if (isMultiline() && preferredMainAxisExtent + childMainAxisMarginBoxExtent > lineBreak && orderedChildren.size() > 0)
             break;
         orderedChildren.append(child);
-        preferredMainAxisExtent += childMainAxisExtent;
+        preferredMainAxisExtent += childMainAxisMarginBoxExtent;
         totalPositiveFlexibility += positiveFlexForChild(child);
         totalNegativeFlexibility += negativeFlexForChild(child);
+
+        LayoutUnit childMinMaxAppliedMainAxisExtent = childMainAxisExtent;
+        // FIXME: valueForLength isn't quite right in quirks mode: percentage heights should check parents until a value is found.
+        // https://bugs.webkit.org/show_bug.cgi?id=81809
+        Length maxLength = isHorizontalFlow() ? child->style()->maxWidth() : child->style()->maxHeight();
+        if (maxLength.isSpecified() && childMinMaxAppliedMainAxisExtent > valueForLength(maxLength, flexboxAvailableContentExtent))
+            childMinMaxAppliedMainAxisExtent = valueForLength(maxLength, flexboxAvailableContentExtent);
+        Length minLength = isHorizontalFlow() ? child->style()->minWidth() : child->style()->minHeight();
+        if (minLength.isSpecified() && childMinMaxAppliedMainAxisExtent < valueForLength(minLength, flexboxAvailableContentExtent))
+            childMinMaxAppliedMainAxisExtent = valueForLength(minLength, flexboxAvailableContentExtent);
+        minMaxAppliedMainAxisExtent += childMinMaxAppliedMainAxisExtent - childMainAxisExtent + childMainAxisMarginBoxExtent;
     }
     return true;
 }
 
 // Returns true if we successfully ran the algorithm and sized the flex items.
-bool RenderFlexibleBox::runFreeSpaceAllocationAlgorithm(const OrderedFlexItemList& children, LayoutUnit& availableFreeSpace, float& totalPositiveFlexibility, float& totalNegativeFlexibility, InflexibleFlexItemSize& inflexibleItems, WTF::Vector<LayoutUnit>& childSizes)
+bool RenderFlexibleBox::resolveFlexibleLengths(FlexSign, const OrderedFlexItemList& children, LayoutUnit& availableFreeSpace, float& totalPositiveFlexibility, float& totalNegativeFlexibility, InflexibleFlexItemSize& inflexibleItems, WTF::Vector<LayoutUnit>& childSizes)
 {
     childSizes.clear();

Modified: trunk/Source/WebCore/rendering/RenderFlexibleBox.h (111595 => 111596)


--- trunk/Source/WebCore/rendering/RenderFlexibleBox.h	2012-03-21 19:42:51 UTC (rev 111595)
+++ trunk/Source/WebCore/rendering/RenderFlexibleBox.h	2012-03-21 20:00:22 UTC (rev 111596)
@@ -49,6 +49,11 @@
     bool isHorizontalFlow() const;
 
 private:
+    enum FlexSign {
+        PositiveFlexibility,
+        NegativeFlexibility,
+    };
+
     struct FlexOrderHashTraits;
     typedef HashSet<int, DefaultHash<int>::Hash, FlexOrderHashTraits> FlexOrderHashSet;
 
@@ -104,9 +109,9 @@
 
     void computeMainAxisPreferredSizes(bool relayoutChildren, FlexOrderHashSet&);
     LayoutUnit lineBreakLength();
-    bool computeNextFlexLine(FlexOrderIterator&, OrderedFlexItemList& orderedChildren, LayoutUnit& preferredMainAxisExtent, float& totalPositiveFlexibility, float& totalNegativeFlexibility);
+    bool computeNextFlexLine(FlexOrderIterator&, OrderedFlexItemList& orderedChildren, LayoutUnit& preferredMainAxisExtent, float& totalPositiveFlexibility, float& totalNegativeFlexibility, LayoutUnit& minMaxAppliedMainAxisExtent);
     LayoutUnit computeAvailableFreeSpace(LayoutUnit preferredMainAxisExtent);
-    bool runFreeSpaceAllocationAlgorithm(const OrderedFlexItemList&, LayoutUnit& availableFreeSpace, float& totalPositiveFlexibility, float& totalNegativeFlexibility, InflexibleFlexItemSize&, WTF::Vector<LayoutUnit>& childSizes);
+    bool resolveFlexibleLengths(FlexSign, const OrderedFlexItemList&, LayoutUnit& availableFreeSpace, float& totalPositiveFlexibility, float& totalNegativeFlexibility, InflexibleFlexItemSize&, WTF::Vector<LayoutUnit>& childSizes);
     void setLogicalOverrideSize(RenderBox* child, LayoutUnit childPreferredSize);
     void prepareChildForPositionedLayout(RenderBox* child, LayoutUnit mainAxisOffset, LayoutUnit crossAxisOffset);
     void layoutAndPlaceChildren(LayoutUnit& crossAxisOffset, const OrderedFlexItemList&, const WTF::Vector<LayoutUnit>& childSizes, LayoutUnit availableFreeSpace);

_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to