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

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

Issue 2456193002: [LayoutNG] Remove last NGConstraintSpace constructor. (Closed)
Patch Set: address comment. Created 4 years, 2 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
« no previous file with comments | « third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/layout/ng/ng_length_utils_test.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_length_utils_test.cc b/third_party/WebKit/Source/core/layout/ng/ng_length_utils_test.cc
index ec9e9922283d035bf0b23c4972216882afe36a31..5f88d702cb30b92217112f73f109d1a3afb91792 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_length_utils_test.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_length_utils_test.cc
@@ -27,12 +27,12 @@ class NGLengthUtilsTest : public ::testing::Test {
int block_size,
bool fixed_inline = false,
bool fixed_block = false) {
- NGConstraintSpace* derived_constraint_space = new NGConstraintSpace(
+ return new NGConstraintSpace(
HorizontalTopBottom, LeftToRight,
- NGLogicalSize(LayoutUnit(inline_size), LayoutUnit(block_size)));
- derived_constraint_space->SetOverflowTriggersScrollbar(false, false);
- derived_constraint_space->SetFixedSize(fixed_inline, fixed_block);
- return derived_constraint_space;
+ new NGPhysicalConstraintSpace(
+ NGPhysicalSize(LayoutUnit(inline_size), LayoutUnit(block_size)),
+ fixed_inline, fixed_block, false, false, FragmentNone, FragmentNone,
+ false));
}
LayoutUnit ResolveInlineLength(
« no previous file with comments | « third_party/WebKit/Source/core/layout/ng/ng_constraint_space_test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698