Use only LayoutText::AbsoluteQuadsForRange() in VisibleUnit::ComputeTextBounds()

third_party/WebKit/Source/core/editing/VisibleUnits.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
  * reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 4087 matching lines @@
 }
 
 Position SkipWhitespace(const Position& position) {
   return SkipWhitespaceAlgorithm(position);
 }
 
 PositionInFlatTree SkipWhitespace(const PositionInFlatTree& position) {
   return SkipWhitespaceAlgorithm(position);
 }
 
-static void CollectAbsoluteBoundsForRange(unsigned start,
-                                          unsigned end,
-                                          const LayoutText& layout_text,
-                                          Vector<IntRect>& rects) {
-  layout_text.AbsoluteRectsForRange(rects, start, end);
-}
-
-static void CollectAbsoluteBoundsForRange(unsigned start,
-                                          unsigned end,
-                                          const LayoutText& layout_text,
-                                          Vector<FloatQuad>& quads) {
-  layout_text.AbsoluteQuadsForRange(quads, start, end);
-}
-
-template <typename RectType, typename Strategy>
-static Vector<RectType> ComputeTextBounds(
+template <typename Strategy>
+static Vector<FloatQuad> ComputeTextBounds(
     const EphemeralRangeTemplate<Strategy>& range) {
   const PositionTemplate<Strategy>& start_position = range.StartPosition();
   const PositionTemplate<Strategy>& end_position = range.EndPosition();
   Node* const start_container = start_position.ComputeContainerNode();
   DCHECK(start_container);
   Node* const end_container = end_position.ComputeContainerNode();
   DCHECK(end_container);
 
-  Vector<RectType> result;
+  Vector<FloatQuad> result;
   for (const Node& node : range.Nodes()) {
     LayoutObject* const layout_object = node.GetLayoutObject();
     if (!layout_object || !layout_object->IsText())
       continue;
     const LayoutText* layout_text = ToLayoutText(layout_object);
     unsigned start_offset =
         node == start_container ? start_position.OffsetInContainerNode() : 0;
     unsigned end_offset = node == end_container
                               ? end_position.OffsetInContainerNode()
                               : std::numeric_limits<unsigned>::max();
-    CollectAbsoluteBoundsForRange(start_offset, end_offset, *layout_text,
-                                  result);
+    layout_text->AbsoluteQuadsForRange(result, start_offset, end_offset);
   }
   return result;
 }
 
 template <typename Strategy>
-static IntRect ComputeTextRectTemplate(
+static FloatRect ComputeTextRectTemplate(
     const EphemeralRangeTemplate<Strategy>& range) {
-  IntRect result;
-  const Vector<IntRect>& rects = ComputeTextBounds<IntRect, Strategy>(range);
-  for (const IntRect& rect : rects)
-    result.Unite(rect);
+  FloatRect result;
+  for (auto rect : ComputeTextBounds<Strategy>(range))
+    result.Unite(rect.BoundingBox());
   return result;
 }
 
 IntRect ComputeTextRect(const EphemeralRange& range) {
-  return ComputeTextRectTemplate(range);
+  return EnclosingIntRect(ComputeTextRectTemplate(range));
 }
 
 IntRect ComputeTextRect(const EphemeralRangeInFlatTree& range) {
-  return ComputeTextRectTemplate(range);
+  return EnclosingIntRect(ComputeTextRectTemplate(range));
 }
 
 Vector<FloatQuad> ComputeTextQuads(const EphemeralRange& range) {
-  return ComputeTextBounds<FloatQuad>(range);
+  return ComputeTextBounds(range);
 }
 
 }  // namespace blink