[LayoutNG] Refactor NGLineBreaker for ShapingLineBreaker

third_party/WebKit/Source/core/layout/ng/inline/ng_inline_item.h

 // Copyright 2017 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.
 
 #ifndef NGInlineItem_h
 #define NGInlineItem_h
 
 #include "platform/LayoutUnit.h"
 #include "platform/fonts/FontFallbackPriority.h"
 #include "platform/fonts/SimpleFontData.h"
@@ skipping 48 matching lines @@
         bidi_level_(UBIDI_LTR),
         shape_options_(kPreContext | kPostContext),
         rotate_sideways_(false),
         fallback_priority_(FontFallbackPriority::kInvalid) {
     DCHECK_GE(end, start);
   }
 
   NGInlineItemType Type() const { return static_cast<NGInlineItemType>(type_); }
   const char* NGInlineItemTypeToString(int val) const;
 
+  const ShapeResult* GetShapeResult() const { return shape_result_.Get(); }

[eae, 2017/05/10 21:31:38]

I've been using the method name TextShapeResult() for this.

[kojii, 2017/05/11 16:17:36]

I like that! ;)

   NGLayoutInlineShapeOptions ShapeOptions() const {
     return static_cast<NGLayoutInlineShapeOptions>(shape_options_);
   }
 
   unsigned StartOffset() const { return start_offset_; }
   unsigned EndOffset() const { return end_offset_; }
   unsigned Length() const { return end_offset_ - start_offset_; }
   TextDirection Direction() const {
     return BidiLevel() & 1 ? TextDirection::kRtl : TextDirection::kLtr;
   }
@@ skipping 86 matching lines @@
 
  private:
   NGInlineItem* start_item_;
   unsigned size_;
   unsigned start_index_;
 };
 
 }  // namespace blink
 
 #endif  // NGInlineItem_h