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

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

Issue 2636353002: [LayoutNG] Remove the state machine from ng_out_of_flow_layout_part. (Closed)
Patch Set: rebase. 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_block_node.cc
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_block_node.cc b/third_party/WebKit/Source/core/layout/ng/ng_block_node.cc
index da6056433edb14f03ab43f58e1a0d2b80b0da18a..c4e60f35eea51a3a6d9c6b912101987b557e4f0c 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_block_node.cc
+++ b/third_party/WebKit/Source/core/layout/ng/ng_block_node.cc
@@ -39,6 +39,12 @@ NGBlockNode::NGBlockNode(ComputedStyle* style)
// included from a compilation unit that lacks the ComputedStyle definition.
NGBlockNode::~NGBlockNode() {}
+void NGBlockNode::LayoutSync(NGConstraintSpace* constraint_space,
+ NGFragment** out) {
+ while (!Layout(constraint_space, out))
+ continue;
+}
+
bool NGBlockNode::Layout(NGConstraintSpace* constraint_space,
NGFragment** out) {
DCHECK(!minmax_algorithm_)
@@ -79,6 +85,13 @@ void NGBlockNode::UpdateLayoutBox(NGPhysicalBoxFragment* fragment,
}
}
+MinAndMaxContentSizes NGBlockNode::ComputeMinAndMaxContentSizesSync() {
+ MinAndMaxContentSizes sizes;
+ while (!ComputeMinAndMaxContentSizes(&sizes))
+ continue;
+ return sizes;
+}
+
bool NGBlockNode::ComputeMinAndMaxContentSizes(MinAndMaxContentSizes* sizes) {
if (!CanUseNewLayout()) {
DCHECK(layout_box_);

Powered by Google App Engine
This is Rietveld 408576698