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

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

Issue 2610253004: Migrate WTF::Vector::append() to ::push_back() [part 9 of N] (Closed)
Patch Set: rebase Created 3 years, 11 months 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 80694bd982443f9d60492837e8a5db53238e33c2..e859346c655bc793bf52353ddae4f599a063557c 100644
--- a/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp
@@ -963,7 +963,7 @@ void LayoutFlexibleBox::layoutFlexItems(bool relayoutChildren,
continue;
}
- allItems.append(constructFlexItem(*child, layoutType));
+ allItems.push_back(constructFlexItem(*child, layoutType));
}
const LayoutUnit lineBreakLength = mainAxisContentExtent(LayoutUnit::max());
@@ -1420,7 +1420,7 @@ void LayoutFlexibleBox::freezeInflexibleItems(FlexSign flexSign,
(flexSign == NegativeFlexibility &&
flexItem.flexBaseContentSize < flexItem.hypotheticalMainContentSize)) {
flexItem.flexedContentSize = flexItem.hypotheticalMainContentSize;
- newInflexibleItems.append(&flexItem);
+ newInflexibleItems.push_back(&flexItem);
}
}
freezeViolations(newInflexibleItems, remainingFreeSpace, totalFlexGrow,
@@ -1481,9 +1481,9 @@ bool LayoutFlexibleBox::resolveFlexibleLengths(
LayoutUnit violation = adjustedChildSize - childSize;
if (violation > 0)
- minViolations.append(&flexItem);
+ minViolations.push_back(&flexItem);
else if (violation < 0)
- maxViolations.append(&flexItem);
+ maxViolations.push_back(&flexItem);
totalViolation += violation;
}
@@ -1892,8 +1892,8 @@ void LayoutFlexibleBox::layoutAndPlaceChildren(
if (m_numberOfInFlowChildrenOnFirstLine == -1)
m_numberOfInFlowChildrenOnFirstLine = children.size();
- lineContexts.append(LineContext(crossAxisOffset, maxChildCrossAxisExtent,
- maxAscent, std::move(children)));
+ lineContexts.push_back(LineContext(crossAxisOffset, maxChildCrossAxisExtent,
+ maxAscent, std::move(children)));
crossAxisOffset += maxChildCrossAxisExtent;
}
@@ -2063,7 +2063,7 @@ void LayoutFlexibleBox::alignChildren(const Vector<LineContext>& lineContexts) {
offset);
}
}
- minMarginAfterBaselines.append(minMarginAfterBaseline);
+ minMarginAfterBaselines.push_back(minMarginAfterBaseline);
}
if (style()->flexWrap() != FlexWrapReverse)
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutBox.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutFlowThread.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698