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

Unified Diff: third_party/WebKit/Source/core/layout/ng/inline/ng_line_breaker_test.cc

Issue 2931563002: [LayoutNG] Implement 'text-align-last' (Closed)
Patch Set: Rebase Created 3 years, 6 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/ng/inline/ng_line_breaker_test.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/inline/ng_line_breaker_test.cc b/third_party/WebKit/Source/core/layout/ng/inline/ng_line_breaker_test.cc
index cd39ff4dff3d66a0ea0b526024a2e312d0c1426d..9332247a44dc446d6d37098051ec84a8a25c4cea 100644
--- a/third_party/WebKit/Source/core/layout/ng/inline/ng_line_breaker_test.cc
+++ b/third_party/WebKit/Source/core/layout/ng/inline/ng_line_breaker_test.cc
@@ -38,13 +38,9 @@ class NGLineBreakerTest : public NGBaseLayoutAlgorithmTest {
NGLineBreaker line_breaker(node, space.Get());
NGInlineLayoutAlgorithm algorithm(node, space.Get());
Vector<NGInlineItemResults> lines;
- while (true) {
- NGInlineItemResults item_results;
- line_breaker.NextLine(&item_results, &algorithm);
- if (item_results.IsEmpty())
- break;
- lines.push_back(item_results);
- }
+ NGLineInfo line_info;
+ while (line_breaker.NextLine(&line_info, &algorithm))
+ lines.push_back(std::move(line_info.Results()));
return lines;
}
};

Powered by Google App Engine
This is Rietveld 408576698