[MarkersIntersectingRange #1] Add DocumentMarkerList::MarkersIntersectingRange()

third_party/WebKit/Source/core/editing/markers/DocumentMarkerListEditor.h

Index: third_party/WebKit/Source/core/editing/markers/DocumentMarkerListEditor.h
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerListEditor.h b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerListEditor.h
index f0ba08f2f806a96d2763fa996070526567f884c5..ec2bb76b466b6992e2867828a7293bbc8610f56e 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerListEditor.h
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerListEditor.h
@@ -45,6 +45,11 @@ class CORE_EXPORT DocumentMarkerListEditor final {
                                              unsigned offset,
                                              unsigned old_length,
                                              unsigned new_length);
+
+  static HeapVector<Member<DocumentMarker>> MarkersIntersectingRange(

[rlanday, 2017/06/22 01:59:37]

This isn't actually an editing method, but I wasn't sure where else to put this.

I propose we rename this class to something like SortedDocumentMarkerListUtils.
All of the methods in this class only work with DocumentMarkerList
implementations that store non-overlapping markers in sorted order, which isn't
otherwise clear, and we have this non-editing method we need to put somewhere.

[yosin_UTC9, 2017/06/22 02:28:20]

I think "Editor" is better than "Utils", actually it should be "Utilities" due
by non-abbreviation rule in Blink, with 
 - Editor can do non-modification operation, e.g. find, move cursor, select,
etc.
 - Editor is shorter than "Utilities".

[Xiaocheng, 2017/06/22 03:06:06]

I'm a little skeptical about this function returning a new vector instead of a
range in the existing vector.

Too bad that WTF::Vector doesn't have a subrange class...

Should we add one?

+      const MarkerList&,
+      unsigned start_offset,
+      unsigned end_offset);
 };
 
 }  // namespace blink