[MarkersIntersectingRange #2.1] Add DocumentMarkerController::FirstMarkerIntersectingOffsetRange()

third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
@@ skipping 380 matching lines @@
 
     const HeapVector<Member<DocumentMarker>>& results =
         list->MarkersIntersectingRange(offset, offset);
     if (!results.IsEmpty())
       return results.front();
   }
 
   return nullptr;
 }
 
+DocumentMarker* DocumentMarkerController::FirstMarkerIntersectingOffsetRange(
+    const Text& node,
+    unsigned start_offset,
+    unsigned end_offset,
+    DocumentMarker::MarkerTypes types) {
+  if (!PossiblyHasMarkers(types))
+    return nullptr;
+
+  MarkerLists* const markers = markers_.at(&node);
+  if (!markers)
+    return nullptr;
+
+  for (DocumentMarker::MarkerType type : types) {
+    const DocumentMarkerList* const list = ListForType(markers, type);
+    if (!list)
+      continue;
+
+    const DocumentMarkerVector& markers_from_this_list =
+        list->MarkersIntersectingRange(start_offset, end_offset);

[yosin_UTC9, 2017/07/20 01:07:01]

We should have DocumentMarjerList::FirstMarkerIntersectingOffsetRange() instead
of
taking first one from list of markers. Except for TextSuggestion marker, we
don't
need to have list of markes intersecting range.

+    if (!markers_from_this_list.IsEmpty())
+      return markers_from_this_list.front();
+  }
+
+  return nullptr;
+}
+
 DocumentMarkerVector DocumentMarkerController::MarkersFor(
     Node* node,
     DocumentMarker::MarkerTypes marker_types) {
   DocumentMarkerVector result;
   if (!PossiblyHasMarkers(marker_types))
     return result;
 
   MarkerLists* markers = markers_.at(node);
   if (!markers)
     return result;
@@ skipping 343 matching lines @@
 }
 
 }  // namespace blink
 
 #ifndef NDEBUG
 void showDocumentMarkers(const blink::DocumentMarkerController* controller) {
   if (controller)
     controller->ShowMarkers();
 }
 #endif