[DMC #12] Move some method impls from DocumentMarkerController.cpp to final place

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

 // 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.
 
 #include "core/editing/markers/TextMatchMarkerListImpl.h"
 
+#include "core/dom/Node.h"
+#include "core/dom/Range.h"
+#include "core/editing/EphemeralRange.h"
 #include "core/editing/markers/DocumentMarkerListEditor.h"
 #include "core/editing/markers/RenderedDocumentMarker.h"
+#include "third_party/WebKit/Source/core/editing/VisibleUnits.h"
 
 namespace blink {
 
 DocumentMarker::MarkerType TextMatchMarkerListImpl::MarkerType() const {
   return DocumentMarker::kTextMatch;
 }
 
 bool TextMatchMarkerListImpl::IsEmpty() const {
   return markers_.IsEmpty();
 }
@@ skipping 27 matching lines @@
                                            unsigned new_length) {
   return DocumentMarkerListEditor::ShiftMarkers(&markers_, offset, old_length,
                                                 new_length);
 }
 
 DEFINE_TRACE(TextMatchMarkerListImpl) {
   visitor->Trace(markers_);
   DocumentMarkerList::Trace(visitor);
 }
 
+Vector<IntRect> TextMatchMarkerListImpl::RenderedRects(const Node& node) {
+  Vector<IntRect> result;
+
+  for (DocumentMarker* marker : markers_) {
+    RenderedDocumentMarker* const rendered_marker =
+        ToRenderedDocumentMarker(marker);
+    UpdateMarkerRenderedRectIfNeeded(node, *rendered_marker);
+    if (!rendered_marker->IsRendered())
+      continue;
+    result.push_back(rendered_marker->RenderedRect());
+  }
+
+  return result;
+}
+
+void TextMatchMarkerListImpl::UpdateMarkerRenderedRectIfNeeded(
+    const Node& node,
+    RenderedDocumentMarker& marker) {
+  if (marker.IsValid())
+    return;
+
+  const Position startPosition(&const_cast<Node&>(node), marker.StartOffset());

[yosin_UTC9, 2017/05/24 09:21:50]

Is it useful to have DomcumentMaker::AsRange(const Node&) -> EphemeralRange?
WDYT?

[rlanday, 2017/05/24 20:25:06]

Might be useful in a few places. I still wish DocumentMarker knew which Node it
was associated with so then we could do stuff like have AsRange() without having
to pass a Node, and have MarkersInRange() with a clean API. But I suppose it
might not be worth the added memory usage/code complexity when moving
DocumentMarkers between Nodes.


Also, I just noticed, in this code I'm moving, I think we should have
start_position and end_position, not startPosition and endPosition

+  const Position endPostion(&const_cast<Node&>(node), marker.EndOffset());
+  EphemeralRange range(startPosition, endPostion);
+  marker.SetRenderedRect(LayoutRect(ComputeTextRect(range)));
+}
+
 }  // namespace blink