[DMC #8] Rename DocumentMarkerController::RenderedRectsForMarkers()

third_party/WebKit/Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 2665 matching lines @@
   return page && page->GetFocusController().IsActive();
 }
 
 void FrameView::InvalidatePaintForTickmarks() {
   if (Scrollbar* scrollbar = VerticalScrollbar())
     scrollbar->SetNeedsPaintInvalidation(
         static_cast<ScrollbarPart>(~kThumbPart));
 }
 
 void FrameView::GetTickmarks(Vector<IntRect>& tickmarks) const {
-  if (!tickmarks_.IsEmpty())
+  if (!tickmarks_.IsEmpty()) {

[yosin_UTC9, 2017/05/19 06:15:59]

nit: Could you use early-return style?

if (!tickmarks_.IsEmpty()) {
  tickmarks = tickmarks_;
  return;
}
tickmarks =
GetFrame().GetDocument()->Markers().RenderedRectsForTextMatchMarkers();

     tickmarks = tickmarks_;
-  else
-    tickmarks = GetFrame().GetDocument()->Markers().RenderedRectsForMarkers(
-        DocumentMarker::kTextMatch);
+  } else {
+    tickmarks =
+        GetFrame().GetDocument()->Markers().RenderedRectsForTextMatchMarkers();
+  }
 }
 
 void FrameView::SetInputEventsTransformForEmulation(
     const IntSize& offset,
     float content_scale_factor) {
   input_events_offset_for_emulation_ = offset;
   input_events_scale_factor_for_emulation_ = content_scale_factor;
 }
 
 IntSize FrameView::InputEventsOffsetForEmulation() const {
@@ skipping 2724 matching lines @@
 void FrameView::SetAnimationHost(
     std::unique_ptr<CompositorAnimationHost> host) {
   animation_host_ = std::move(host);
 }
 
 LayoutUnit FrameView::CaretWidth() const {
   return LayoutUnit(GetChromeClient()->WindowToViewportScalar(1));
 }
 
 }  // namespace blink