[LayoutNG] Remove the ng_layout_coordinator and temporary LayoutSync method.

third_party/WebKit/Source/core/layout/ng/ng_block_layout_algorithm.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/layout/ng/ng_block_layout_algorithm.h"
 
 #include "core/layout/ng/ng_absolute_utils.h"
 #include "core/layout/ng/ng_block_break_token.h"
 #include "core/layout/ng/ng_box_fragment.h"
 #include "core/layout/ng/ng_column_mapper.h"
@@ skipping 201 matching lines @@
         ComputeMinAndMaxContentContribution(*node->Style(), child_minmax);
 
     sizes->min_content = std::max(sizes->min_content, child_sizes.min_content);
     sizes->max_content = std::max(sizes->max_content, child_sizes.max_content);
   }
 
   sizes->max_content = std::max(sizes->min_content, sizes->max_content);
   return true;
 }
 
-NGLayoutStatus NGBlockLayoutAlgorithm::Layout(
-    NGPhysicalFragment* child_fragment,
-    NGPhysicalFragment** fragment_out,
-    NGLayoutAlgorithm** algorithm_out) {
+NGPhysicalFragment* NGBlockLayoutAlgorithm::Layout() {
   WTF::Optional<MinAndMaxContentSizes> sizes;
   if (NeedMinAndMaxContentSizes(ConstraintSpace(), Style())) {
     // TODO(ikilpatrick): Change ComputeMinAndMaxContentSizes to return
     // MinAndMaxContentSizes.
     sizes = MinAndMaxContentSizes();
     ComputeMinAndMaxContentSizes(&*sizes);
   }
 
   border_and_padding_ =
       ComputeBorders(Style()) + ComputePadding(ConstraintSpace(), Style());
@@ skipping 48 matching lines @@
       builder_->AddOutOfFlowChildCandidate(current_child_,
                                            GetChildSpaceOffset());
       current_child_ = current_child_->NextSibling();
       continue;
     }
 
     DCHECK(!ConstraintSpace().HasBlockFragmentation() ||
            SpaceAvailableForCurrentChild() > LayoutUnit());
     space_for_current_child_ = CreateConstraintSpaceForCurrentChild();
 
-    NGFragment* fragment;
-    current_child_->LayoutSync(space_for_current_child_, &fragment);
-    NGPhysicalFragment* child_fragment = fragment->PhysicalFragment();
-
-    // TODO(layout_ng): Seems like a giant hack to call this here.
-    current_child_->UpdateLayoutBox(toNGPhysicalBoxFragment(child_fragment),

[cbiesinger, 2017/01/20 21:21:56]

Oh I see... this was only needed because we did not call box::Layout... wow,
glad we got rid of that.

[ikilpatrick, 2017/01/20 23:19:57]

Yeah going through the boxes now might make things easier.

-                                    space_for_current_child_);
+    NGPhysicalFragment* child_fragment =
+        current_child_->Layout(space_for_current_child_);
 
     FinishCurrentChildLayout(new NGBoxFragment(
         ConstraintSpace().WritingMode(), ConstraintSpace().Direction(),
         toNGPhysicalBoxFragment(child_fragment)));
 
     if (!ProceedToNextUnfinishedSibling(child_fragment))
       break;
   }
 
   content_size_ += border_and_padding_.block_end;
@@ skipping 11 matching lines @@
   current_child_ = nullptr;
 
   while (!LayoutOutOfFlowChild())
     continue;
 
   builder_->SetInlineOverflow(max_inline_size_).SetBlockOverflow(content_size_);
 
   if (ConstraintSpace().HasBlockFragmentation())
     FinalizeForFragmentation();
 
-  *fragment_out = builder_->ToBoxFragment();
-  return kNewFragment;
+  return builder_->ToBoxFragment();
 }
 
 void NGBlockLayoutAlgorithm::FinishCurrentChildLayout(NGFragment* fragment) {
   NGBoxStrut child_margins = ComputeMargins(
       *space_for_current_child_, CurrentChildStyle(),
       constraint_space_->WritingMode(), constraint_space_->Direction());
 
   NGLogicalOffset fragment_offset;
   if (CurrentChildStyle().isFloating()) {
     fragment_offset = PositionFloatFragment(*fragment, child_margins);
@@ skipping 334 matching lines @@
   visitor->trace(builder_);
   visitor->trace(space_builder_);
   visitor->trace(space_for_current_child_);
   visitor->trace(current_child_);
   visitor->trace(out_of_flow_layout_);
   visitor->trace(out_of_flow_candidates_);
   visitor->trace(fragmentainer_mapper_);
 }
 
 }  // namespace blink