[DMC #11] Move some rendered rect method impls from DMC To TextMatchMarkerListImpl

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

Index: third_party/WebKit/Source/core/editing/markers/TextMatchMarkerListImpl.h
diff --git a/third_party/WebKit/Source/core/editing/markers/TextMatchMarkerListImpl.h b/third_party/WebKit/Source/core/editing/markers/TextMatchMarkerListImpl.h
index ca4ae45f27b214b1bd72e2f1adf01ea3ae27bd8a..e46655de354a23e53219bd0606adcdca4288cf58 100644
--- a/third_party/WebKit/Source/core/editing/markers/TextMatchMarkerListImpl.h
+++ b/third_party/WebKit/Source/core/editing/markers/TextMatchMarkerListImpl.h
@@ -9,6 +9,9 @@
 
 namespace blink {
 
+class IntRect;
+class RenderedDocumentMarker;
+
 // Implementation of DocumentMarkerList for TextMatch markers.
 // Markers are kept sorted by start offset, under the assumption that
 // TextMatch markers are typically inserted in an order.
@@ -33,6 +36,10 @@ class CORE_EXPORT TextMatchMarkerListImpl final : public DocumentMarkerList {
                     unsigned new_length) final;
   DECLARE_VIRTUAL_TRACE();
 
+  // TextMatchMarkerListImpl-specific
+  void AppendRenderedRectsToVector(const Node&, Vector<IntRect>*);

[yosin_UTC9, 2017/05/22 05:24:18]

Let's make this function to return |Vector<IntRect>|, c++11-ish, rather than
updating parameter.

+  void UpdateMarkerRenderedRectIfNeeded(const Node&, RenderedDocumentMarker&);

[yosin_UTC9, 2017/05/22 05:24:18]

nit: |UpdateMarkerRenderedRectIfNeeded()| should be in private section.
It is called only from |AppendRenderedRectsToVector()|

+
  private:
   HeapVector<Member<DocumentMarker>> markers_;