[LayoutNG] Remove the state machine from ng_out_of_flow_layout_part.

third_party/WebKit/Source/core/layout/ng/ng_block_node.h

Index: third_party/WebKit/Source/core/layout/ng/ng_block_node.h
diff --git a/third_party/WebKit/Source/core/layout/ng/ng_block_node.h b/third_party/WebKit/Source/core/layout/ng/ng_block_node.h
index d2b9a3835e0bbb53fab395731e5f97d1974099b0..e7d7f65dc6ad1c5d97ad36325f5491af978eee77 100644
--- a/third_party/WebKit/Source/core/layout/ng/ng_block_node.h
+++ b/third_party/WebKit/Source/core/layout/ng/ng_block_node.h
@@ -34,6 +34,8 @@ class CORE_EXPORT NGBlockNode final : public NGLayoutInputNode {
   ~NGBlockNode() override;
 
   bool Layout(NGConstraintSpace*, NGFragment**) override;
+  void LayoutSync(NGConstraintSpace*, NGFragment**);
+

[atotic, 2017/01/18 18:48:49]

Are Sync methods temporary?

What are our plans for interruptible code? We've thrown out resumable, but
interruptible is still required.

[ikilpatrick, 2017/01/18 19:32:13]

Yup the sync methods are temporary, the eventual Layout() method will be:

NGFragment* Layout(NGConstraintSpace*, NGBreakToken*);

Returning a nullptr from Layout will mean that the parent algorithm should
abort.

Inside each of the algorithms they'll have a macro which does this effectively:

NGLayoutAlgorithm::Layout(/* arguments */) {
  for (child of children) {
    fragment = NG_MAYBE_ABORT(child->Layout(child_space, break_token));
}

NG_MAYBE_ABORT(fragment) {
  if (!fragment) return nullptr;
  if (OutOfTime()) return nullptr;
  return fragment;
}

   NGBlockNode* NextSibling() override;
 
   // Computes the value of min-content and max-content for this box.
@@ -46,6 +48,7 @@ class CORE_EXPORT NGBlockNode final : public NGLayoutInputNode {
   // often enough that it returns true before calling
   // ComputeOrSynthesizeMinAndMaxContentSizes)
   bool ComputeMinAndMaxContentSizes(MinAndMaxContentSizes*);
+  MinAndMaxContentSizes ComputeMinAndMaxContentSizesSync();
 
   const ComputedStyle* Style() const;
   ComputedStyle* MutableStyle();