[LayoutNG] Refactor NGLineBreaker for ShapingLineBreaker

third_party/WebKit/Source/core/layout/ng/inline/ng_inline_node_test.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/inline/ng_inline_node.h"
 
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/ng/inline/ng_inline_layout_algorithm.h"
 #include "core/layout/ng/inline/ng_physical_line_box_fragment.h"
 #include "core/layout/ng/inline/ng_physical_text_fragment.h"
@@ skipping 295 matching lines @@
   TEST_TEXT_FRAGMENT(fragments[0], node, 0u, 0u, 6u, TextDirection::kLtr);
   TEST_TEXT_FRAGMENT(fragments[1], node, 6u, 16u, 21u, TextDirection::kRtl);
   TEST_TEXT_FRAGMENT(fragments[2], node, 4u, 14u, 15u, TextDirection::kLtr);
   TEST_TEXT_FRAGMENT(fragments[3], node, 2u, 7u, 13u, TextDirection::kRtl);
   TEST_TEXT_FRAGMENT(fragments[4], node, 8u, 22u, 28u, TextDirection::kLtr);
 }
 
 TEST_F(NGInlineNodeTest, MinMaxContentSize) {
   UseLayoutObjectAndAhem();
   NGInlineNodeForTest* node = CreateInlineNode();
-  node->Append("AB CDE", style_.Get(), layout_object_);
+  node->Append("AB CDEF", style_.Get(), layout_object_);
   node->ShapeText();
   MinMaxContentSize sizes = node->ComputeMinMaxContentSize();
-  // TODO(kojii): min_content should be 20, but is 30 until
-  // NGInlineLayoutAlgorithm implements trailing spaces correctly.
-  EXPECT_EQ(30, sizes.min_content);
-  EXPECT_EQ(60, sizes.max_content);
+  EXPECT_EQ(40, sizes.min_content);
+  EXPECT_EQ(70, sizes.max_content);
 }
 
 TEST_F(NGInlineNodeTest, MinMaxContentSizeElementBoundary) {
   UseLayoutObjectAndAhem();
   NGInlineNodeForTest* node = CreateInlineNode();
   node->Append("A B", style_.Get(), layout_object_);
   node->Append("C D", style_.Get(), layout_object_);
   node->ShapeText();
   MinMaxContentSize sizes = node->ComputeMinMaxContentSize();
   // |min_content| should be the width of "BC" because there is an element
   // boundary between "B" and "C" but no break opportunities.
-  // TODO(kojii): min_content should be 20, but is 30 until
-  // NGInlineLayoutAlgorithm implements trailing spaces correctly.
-  EXPECT_EQ(30, sizes.min_content);
+  EXPECT_EQ(20, sizes.min_content);
   EXPECT_EQ(60, sizes.max_content);
 }
 
 }  // namespace blink