[LayoutNG] Don't pass parent NGFragmentBuilder into NGLineBuilder.

third_party/WebKit/Source/core/layout/ng/ng_line_builder.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_line_builder.h"
 
 #include "core/layout/BidiRun.h"
 #include "core/layout/LayoutBlockFlow.h"
 #include "core/layout/line/LineInfo.h"
 #include "core/layout/line/RootInlineBox.h"
 #include "core/layout/ng/ng_bidi_paragraph.h"
 #include "core/layout/ng/ng_box_fragment.h"
 #include "core/layout/ng/ng_constraint_space.h"
 #include "core/layout/ng/ng_constraint_space_builder.h"
 #include "core/layout/ng/ng_fragment_builder.h"
 #include "core/layout/ng/ng_inline_node.h"
 #include "core/layout/ng/ng_length_utils.h"
 #include "core/layout/ng/ng_text_fragment.h"
 #include "core/style/ComputedStyle.h"
 #include "platform/text/BidiRunList.h"
 
 namespace blink {
 
 NGLineBuilder::NGLineBuilder(NGInlineNode* inline_box,
-                             NGConstraintSpace* constraint_space,
-                             NGFragmentBuilder* containing_block_builder)
+                             NGConstraintSpace* constraint_space)
     : inline_box_(inline_box),
       constraint_space_(constraint_space),
-      containing_block_builder_(containing_block_builder),
       container_builder_(NGPhysicalFragment::kFragmentBox, inline_box_),
       container_layout_result_(nullptr),
       is_horizontal_writing_mode_(
           blink::IsHorizontalWritingMode(constraint_space->WritingMode()))
 #if DCHECK_IS_ON()
       ,
       is_bidi_reordered_(false)
 #endif
 {
   if (!is_horizontal_writing_mode_)
@@ skipping 262 matching lines @@
       // The direction of a fragment is the CSS direction to resolve logical
       // properties, not the resolved bidi direction.
       text_builder.SetDirection(style->direction())
           .SetInlineSize(line_item_chunk.inline_size)
           .SetInlineOverflow(line_item_chunk.inline_size)
           .SetBlockSize(line_height)
           .SetBlockOverflow(line_height);
     } else {
       LayoutObject* layout_object = item.GetLayoutObject();
       if (layout_object->isOutOfFlowPositioned()) {
-        if (containing_block_builder_) {
-          // Absolute positioning blockifies the box's display type.
-          // https://drafts.csswg.org/css-display/#transformations
-          containing_block_builder_->AddOutOfFlowChildCandidate(
-              new NGBlockNode(layout_object),
-              NGLogicalOffset(line_box_data.inline_size, content_size_));
-        }
+        // Absolute positioning blockifies the box's display type.
+        // https://drafts.csswg.org/css-display/#transformations
+        //
+        // TODO(layout-dev): Report the correct static position for the out of
+        // flow descendant. We can't do this here yet as it doesn't know the
+        // size of the line box.
+        container_builder_.AddOutOfFlowDescendant(
+            new NGBlockNode(layout_object),
+            NGStaticPosition::Create(ConstraintSpace().WritingMode(),
+                                     ConstraintSpace().Direction(),
+                                     NGPhysicalOffset()));
         continue;
       } else if (layout_object->isFloating()) {
         // TODO(kojii): Implement float.
         DLOG(ERROR) << "Floats in inline not implemented yet.";
         // TODO(kojii): Temporarily clearNeedsLayout() for not to assert.
         layout_object->clearNeedsLayout();
         continue;
       }
       block_start = PlaceAtomicInline(item, estimated_baseline, &line_box_data,
                                       &text_builder);
@@ skipping 230 matching lines @@
         baseline_position - LayoutUnit(line_box_data.max_ascent),
         baseline_position + LayoutUnit(line_box_data.max_descent),
         line_box_data.top_with_leading,
         baseline_position + LayoutUnit(line_box_data.max_descent_and_leading));
 
     bidi_runs.deleteRuns();
     fragments_for_bidi_runs.clear();
   }
 }
 }  // namespace blink