LayoutObject::absoluteBoundingBoxRectForRange() should take EphemeralRange

third_party/WebKit/Source/core/layout/LayoutObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved.
  *               (http://www.torchmobile.com/)
@@ skipping 975 matching lines @@
 
 IntRect LayoutObject::absoluteElementBoundingBoxRect() const {
   Vector<LayoutRect> rects;
   const LayoutBoxModelObject& container = enclosingLayer()->layoutObject();
   addElementVisualOverflowRects(
       rects, LayoutPoint(localToAncestorPoint(FloatPoint(), &container)));
   return container.localToAbsoluteQuad(FloatQuad(FloatRect(unionRect(rects))))
       .enclosingBoundingBox();
 }
 
-FloatRect LayoutObject::absoluteBoundingBoxRectForRange(const Range* range) {
-  if (!range || !range->startContainer())
+FloatRect LayoutObject::absoluteBoundingBoxRectForRange(
+    const EphemeralRange& range) {
+  if (range.isNull() || !range.startPosition().computeContainerNode())
     return FloatRect();
 
-  range->ownerDocument().updateStyleAndLayout();
+  range.document().updateStyleAndLayout();
 
   Vector<FloatQuad> quads;
-  range->textQuads(quads);
+  textQuad(quads, range);

[Xiaocheng, 2017/03/28 21:29:34]

Let's make textQuad return |Vector<FloatQuad>| directly, since WTF::Vector
supports moving semantics.

[tanvir, 2017/03/30 19:11:06]

Done.

 
   FloatRect result;
   for (size_t i = 0; i < quads.size(); ++i)
     result.unite(quads[i].boundingBox());
 
   return result;
 }
 
 void LayoutObject::addAbsoluteRectForLayer(IntRect& result) {
   if (hasLayer())
@@ skipping 2611 matching lines @@
     const blink::LayoutObject* root = object1;
     while (root->parent())
       root = root->parent();
     root->showLayoutTreeAndMark(object1, "*", object2, "-", 0);
   } else {
     WTFLogAlways("%s", "Cannot showLayoutTree. Root is (nil)");
   }
 }
 
 #endif