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

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

Issue 2649583002: [LayoutNG] Remove the ng_layout_coordinator and temporary LayoutSync method. (Closed)
Patch Set: rebase v2 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/ng/ng_inline_layout_algorithm.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.cc b/third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.cc
index 1c8548f06702027845e2af33ff7e3501aedb6116..f6f43aa2a29c354b273c97d9aae3d9df10216c70 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_inline_layout_algorithm.cc
@@ -30,10 +30,7 @@ NGInlineLayoutAlgorithm::NGInlineLayoutAlgorithm(
DCHECK(style_);
}
-NGLayoutStatus NGInlineLayoutAlgorithm::Layout(
- NGPhysicalFragment*,
- NGPhysicalFragment** fragment_out,
- NGLayoutAlgorithm**) {
+NGPhysicalFragment* NGInlineLayoutAlgorithm::Layout() {
// TODO(kojii): Implement sizing and child constraint spaces. Share common
// logic with NGBlockLayoutAlgorithm using composition.
builder_ = new NGFragmentBuilder(NGPhysicalFragment::kFragmentBox);
@@ -52,10 +49,9 @@ NGLayoutStatus NGInlineLayoutAlgorithm::Layout(
}
line_builder_->CreateFragments(builder_);
- *fragment_out = builder_->ToBoxFragment();
+ NGPhysicalFragment* fragment = builder_->ToBoxFragment();
line_builder_->CopyFragmentDataToLayoutBlockFlow();
- state_ = kStateInit;
- return kNewFragment;
+ return fragment;
}
bool NGInlineLayoutAlgorithm::LayoutCurrentChild() {

Powered by Google App Engine
This is Rietveld 408576698