Get rid of the extra vector when setting up the orderIterator.

Source/core/rendering/RenderFlexibleBox.h

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 66 matching lines @@
     class OrderIterator {
         WTF_MAKE_NONCOPYABLE(OrderIterator);
     public:
         OrderIterator(const RenderFlexibleBox*);
         void setOrderValues(Vector<int>&);
         RenderBox* currentChild() const { return m_currentChild; }
         RenderBox* first();
         RenderBox* next();
         void reset();
 
+        Vector<int> m_orderValues;
+
     private:
         const RenderFlexibleBox* m_flexibleBox;
         RenderBox* m_currentChild;
-        Vector<int> m_orderValues;
         Vector<int>::const_iterator m_orderValuesIterator;
     };
 
     typedef HashMap<const RenderBox*, LayoutUnit> InflexibleFlexItemSize;
     typedef Vector<RenderBox*> OrderedFlexItemList;
 
     struct LineContext;
     struct Violation;
 
     // Use an inline capacity of 8, since flexbox containers usually have less than 8 children.
@@ skipping 43 matching lines @@
     bool updateAutoMarginsInCrossAxis(RenderBox* child, LayoutUnit availableAlignmentSpace);
     void repositionLogicalHeightDependentFlexItems(Vector<LineContext>&);
     LayoutUnit clientLogicalBottomAfterRepositioning();
     void appendChildFrameRects(ChildFrameRects&);
     void repaintChildrenDuringLayoutIfMoved(const ChildFrameRects&);
 
     LayoutUnit availableAlignmentSpaceForChild(LayoutUnit lineCrossAxisExtent, RenderBox*);
     LayoutUnit marginBoxAscentForChild(RenderBox*);
 
     LayoutUnit computeChildMarginValue(Length margin, RenderView*);
-    void computeMainAxisPreferredSizes(Vector<int>&);
+    void prepareOrderIteratorAndMargins();
     LayoutUnit adjustChildSizeForMinAndMax(RenderBox*, LayoutUnit childSize);
     bool computeNextFlexLine(OrderedFlexItemList& orderedChildren, LayoutUnit& preferredMainAxisExtent, double& totalFlexGrow, double& totalWeightedFlexShrink, LayoutUnit& minMaxAppliedMainAxisExtent, bool& hasInfiniteLineLength);
 
     bool resolveFlexibleLengths(FlexSign, const OrderedFlexItemList&, LayoutUnit& availableFreeSpace, double& totalFlexGrow, double& totalWeightedFlexShrink, InflexibleFlexItemSize&, Vector<LayoutUnit>& childSizes, bool hasInfiniteLineLength);
     void freezeViolations(const Vector<Violation>&, LayoutUnit& availableFreeSpace, double& totalFlexGrow, double& totalWeightedFlexShrink, InflexibleFlexItemSize&, bool hasInfiniteLineLength);
 
     void resetAutoMarginsAndLogicalTopInCrossAxis(RenderBox*);
     bool needToStretchChild(RenderBox*);
     void setLogicalOverrideSize(RenderBox* child, LayoutUnit childPreferredSize);
     void prepareChildForPositionedLayout(RenderBox* child, LayoutUnit mainAxisOffset, LayoutUnit crossAxisOffset, PositionedLayoutMode);
@@ skipping 21 matching lines @@
     ASSERT_WITH_SECURITY_IMPLICATION(!object || object->isFlexibleBox());
     return static_cast<const RenderFlexibleBox*>(object);
 }
 
 // This will catch anyone doing an unnecessary cast.
 void toRenderFlexibleBox(const RenderFlexibleBox*);
 
 } // namespace WebCore
 
 #endif // RenderFlexibleBox_h