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

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

Issue 2724133003: [LayoutNG] Switch NGConstraintSpace to being RefCounted. (Closed)
Patch Set: moar. Created 3 years, 10 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/ng_inline_node.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_inline_node.cc b/third_party/WebKit/Source/core/layout/ng/ng_inline_node.cc
index 7d5b148457f872d9dcc3b2376805b4e77ce8d841..b94822220d74f2119b34d12730c55fc4cd30675b 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_inline_node.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_inline_node.cc
@@ -268,13 +268,13 @@ MinAndMaxContentSizes NGInlineNode::ComputeMinAndMaxContentSizes() {
// opportunity.
NGWritingMode writing_mode =
FromPlatformWritingMode(BlockStyle()->getWritingMode());
- NGConstraintSpace* constraint_space =
+ RefPtr<NGConstraintSpace> constraint_space =
NGConstraintSpaceBuilder(writing_mode)
.SetTextDirection(BlockStyle()->direction())
.SetAvailableSize({LayoutUnit(), NGSizeIndefinite})
.ToConstraintSpace(writing_mode);
- NGLineBuilder line_builder(this, constraint_space);
- LayoutInline(constraint_space, &line_builder);
+ NGLineBuilder line_builder(this, constraint_space.get());
+ LayoutInline(constraint_space.get(), &line_builder);
MinAndMaxContentSizes sizes;
sizes.min_content = line_builder.MaxInlineSize();

Powered by Google App Engine
This is Rietveld 408576698