In-place change of Range::testRects() and boundingBox() to EphemeralRange.

third_party/WebKit/Source/core/dom/Range.cpp

 /*
  * (C) 1999 Lars Knoll (knoll@kde.org)
  * (C) 2000 Gunnstein Lye (gunnstein@netcom.no)
  * (C) 2000 Frederik Holljen (frederik.holljen@hig.no)
  * (C) 2001 Peter Kelly (pmk@post.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2011 Motorola Mobility. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 1432 matching lines @@
 }
 
 Node* Range::PastLastNode() const {
   if (end_.Container().IsCharacterDataNode())
     return NodeTraversal::NextSkippingChildren(end_.Container());
   if (Node* child = NodeTraversal::ChildAt(end_.Container(), end_.Offset()))
     return child;
   return NodeTraversal::NextSkippingChildren(end_.Container());
 }
 
+// TODO(hs1217.lee):: we should move this implement to VisibleUnits and then
+// this function will remove.
+static Vector<IntRect> computeTextRects(const EphemeralRange&);
+
 IntRect Range::BoundingBox() const {
   IntRect result;
-  Vector<IntRect> rects;
-  TextRects(rects);
+  const Vector<IntRect> rects = computeTextRects(EphemeralRange(this));

[yosin_UTC9, 2017/04/14 01:08:50]

nit: s/Vector<IntRect>/Vector<IntRect>&/

[Hwanseung Lee, 2017/04/15 02:54:33]

Done.

   for (const IntRect& rect : rects)
     result.Unite(rect);
   return result;
 }
 
-void Range::TextRects(Vector<IntRect>& rects, bool use_selection_height) const {

[yosin_UTC9, 2017/04/14 01:08:51]

Note: |use_selection_height| is default to false and all call sites, actually
one, pass |false| as default.
So, we don't need to have it.

[Hwanseung Lee, 2017/04/15 02:54:33]

right. this line was removed.

-  Node* start_container = &start_.Container();
+static Vector<IntRect> computeTextRects(const EphemeralRange& range) {
+  const Position& start_position = range.StartPosition();
+  const Position& end_position = range.EndPosition();
+  Node* start_container = start_position.ComputeContainerNode();

[yosin_UTC9, 2017/04/14 01:08:51]

nit: s/Node*/Node* const/

[Hwanseung Lee, 2017/04/15 02:54:33]

Done.

   DCHECK(start_container);
-  Node* end_container = &end_.Container();
+  Node* end_container = end_position.ComputeContainerNode();

[yosin_UTC9, 2017/04/14 01:08:51]

nit: s/Node*/Node* const/

[Hwanseung Lee, 2017/04/15 02:54:33]

Done.

   DCHECK(end_container);
 
-  Node* stop_node = PastLastNode();
-  for (Node* node = FirstNode(); node != stop_node;
-       node = NodeTraversal::Next(*node)) {
-    LayoutObject* r = node->GetLayoutObject();
-    if (!r || !r->IsText())
+  Vector<IntRect> rects;
+  for (Node& node : range.Nodes()) {

[yosin_UTC9, 2017/04/14 01:08:51]

nit: s/Node&/const Node&/

It is nice to use range-for instead of using |stop_node| and
|NodeTraversal::Next()|.

[Hwanseung Lee, 2017/04/15 02:54:33]

Done.

+    LayoutObject* layoutObject = node.GetLayoutObject();

[yosin_UTC9, 2017/04/14 01:08:51]

nit: s/LayoutObject*/LayoutObject* const/

[Hwanseung Lee, 2017/04/15 02:54:33]

Done.

+    if (!layoutObject || !layoutObject->IsText())
       continue;
-    LayoutText* layout_text = ToLayoutText(r);
-    unsigned start_offset = node == start_container ? start_.Offset() : 0;
+    LayoutText* layout_text = ToLayoutText(layoutObject);
+    unsigned start_offset =
+        node == start_container ? start_position.OffsetInContainerNode() : 0;
     unsigned end_offset = node == end_container
-                              ? end_.Offset()
+                              ? end_position.OffsetInContainerNode()
                               : std::numeric_limits<unsigned>::max();
-    layout_text->AbsoluteRectsForRange(rects, start_offset, end_offset,
-                                       use_selection_height);
+    layout_text->AbsoluteRectsForRange(rects, start_offset, end_offset);

[yosin_UTC9, 2017/04/14 01:08:51]

Note: |LayoutText::AbsoulteRectsForRange()| takes |use_selection_height| as
default parameter to |false|,
so, we don't need to pass it.

   }
+  return rects;
 }
 
 void Range::TextQuads(Vector<FloatQuad>& quads,
                       bool use_selection_height) const {
   Node* start_container = &start_.Container();
   DCHECK(start_container);
   Node* end_container = &end_.Container();
   DCHECK(end_container);
 
   Node* stop_node = PastLastNode();
@@ skipping 336 matching lines @@
                      .Data()
               << "start offset: " << range->startOffset()
               << ", end offset: " << range->endOffset();
   } else {
     LOG(INFO) << "Cannot show tree if range is null, or if boundary points are "
                  "invalid.";
   }
 }
 
 #endif