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

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

Issue 2587283004: [layoutng] Add a shrink-to-fit flag to the constraint space (Closed)
Patch Set: review comments Created 4 years 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/ng_constraint_space.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc b/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc
index 4fb438d917909e5ffe50de41e02cb7bc1464921f..9ec2d52ca1886e40deb9e0569acdd3ca48bad805 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_constraint_space.cc
@@ -18,6 +18,7 @@ NGConstraintSpace::NGConstraintSpace(
NGLogicalSize percentage_resolution_size,
bool is_fixed_size_inline,
bool is_fixed_size_block,
+ bool is_shrink_to_fit,
bool is_inline_direction_triggers_scrollbar,
bool is_block_direction_triggers_scrollbar,
NGFragmentationType block_direction_fragmentation_type,
@@ -27,6 +28,7 @@ NGConstraintSpace::NGConstraintSpace(
percentage_resolution_size_(percentage_resolution_size),
is_fixed_size_inline_(is_fixed_size_inline),
is_fixed_size_block_(is_fixed_size_block),
+ is_shrink_to_fit_(is_shrink_to_fit),
is_inline_direction_triggers_scrollbar_(
is_inline_direction_triggers_scrollbar),
is_block_direction_triggers_scrollbar_(
@@ -78,6 +80,8 @@ NGConstraintSpace* NGConstraintSpace::CreateFromLayoutObject(
box.styleRef().overflowBlockDirection() == EOverflow::Auto)
.SetIsFixedSizeInline(fixed_inline)
.SetIsFixedSizeBlock(fixed_block)
+ .SetIsShrinkToFit(
+ box.sizesLogicalWidthToFitContent(box.styleRef().logicalWidth()))
.SetIsNewFormattingContext(is_new_fc)
.SetTextDirection(box.styleRef().direction())
.ToConstraintSpace();

Powered by Google App Engine
This is Rietveld 408576698