Utilize NGInlineCursor in InspectorCSSAgent::CollectPlatformFontsForLayoutObject()

This patch changes |InspectorCSSAgent::CollectPlatformFontsForLayoutObject()| to
utilize |NGInlineCursor| for preparation of migration to |NGFragmentItem|.

Bug: 982194
Change-Id: Id8550ce0d70def5b1eb1f7d442e0e97152cc7fba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1880360
Commit-Queue: Koji Ishii <kojii@chromium.org>
Auto-Submit: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710677}

third_party/blink/renderer/core/inspector/inspector_css_agent.cc

@@ -78,7 +78,7 @@
 #include "third_party/blink/renderer/core/layout/layout_text.h"
 #include "third_party/blink/renderer/core/layout/layout_view.h"
 #include "third_party/blink/renderer/core/layout/line/inline_text_box.h"
-#include "third_party/blink/renderer/core/layout/ng/inline/ng_physical_text_fragment.h"
+#include "third_party/blink/renderer/core/layout/ng/inline/ng_inline_cursor.h"
 #include "third_party/blink/renderer/core/loader/document_loader.h"
 #include "third_party/blink/renderer/core/page/page.h"
 #include "third_party/blink/renderer/core/paint/ng/ng_paint_fragment.h"
@@ -1185,12 +1185,11 @@ void InspectorCSSAgent::CollectPlatformFontsForLayoutObject(
   LayoutText* layout_text = ToLayoutText(layout_object);
 
   if (RuntimeEnabledFeatures::LayoutNGEnabled()) {
-    auto fragments = NGPaintFragment::InlineFragmentsFor(layout_object);
-    if (fragments.IsInLayoutNGInlineFormattingContext()) {
-      for (const NGPaintFragment* fragment : fragments) {
-        const auto& text_fragment =
-            To<NGPhysicalTextFragment>(fragment->PhysicalFragment());
-        const ShapeResultView* shape_result = text_fragment.TextShapeResult();
+    if (layout_object->IsInLayoutNGInlineFormattingContext()) {
+      NGInlineCursor cursor;
+      cursor.MoveTo(*layout_object);
+      for (; cursor; cursor.MoveToNextForSameLayoutObject()) {
+        const ShapeResultView* shape_result = cursor.CurrentTextShapeResult();
         if (!shape_result)
           continue;
         Vector<ShapeResult::RunFontData> run_font_data_list;

third_party/blink/renderer/core/layout/ng/inline/ng_inline_cursor.cc

@@ -412,6 +412,19 @@ unsigned NGInlineCursor::CurrentTextEndOffset() const {
   return 0u;
 }
 
+const ShapeResultView* NGInlineCursor::CurrentTextShapeResult() const {
+  DCHECK(IsText());
+  if (current_paint_fragment_) {
+    return To<NGPhysicalTextFragment>(
+               current_paint_fragment_->PhysicalFragment())
+        .TextShapeResult();
+  }
+  if (current_item_)
+    return current_item_->TextShapeResult();
+  NOTREACHED();
+  return nullptr;
+}
+
 PhysicalRect NGInlineCursor::CurrentLocalRect(unsigned start_offset,
                                               unsigned end_offset) const {
   DCHECK(IsText());

third_party/blink/renderer/core/layout/ng/inline/ng_inline_cursor.h

@@ -26,6 +26,7 @@ class Node;
 struct PhysicalOffset;
 struct PhysicalRect;
 struct PhysicalSize;
+class ShapeResultView;
 
 // This class traverses fragments in an inline formatting context.
 //
@@ -150,6 +151,10 @@ class CORE_EXPORT NGInlineCursor {
   unsigned CurrentTextStartOffset() const;
   unsigned CurrentTextEndOffset() const;
 
+  // Returns |ShapeResultView| of the current position. It is error to call
+  // other than text.
+  const ShapeResultView* CurrentTextShapeResult() const;
+
   // The layout box of text in (start, end) range in local coordinate.
   // Start and end offsets must be between |CurrentTextStartOffset()| and
   // |CurrentTextEndOffset()|. It is error to call other than text.