Make RenderedRectsForMarkers() to ignore disconnected nodes.

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 1431 matching lines @@
 
 Node* Range::pastLastNode() const {
   if (m_end.container()->isCharacterDataNode())
     return NodeTraversal::nextSkippingChildren(*m_end.container());
   if (Node* child = NodeTraversal::childAt(*m_end.container(), m_end.offset()))
     return child;
   return NodeTraversal::nextSkippingChildren(*m_end.container());
 }
 
 IntRect Range::boundingBox() const {
+  return computeBoundingBox(EphemeralRange(this));
+}
+
+template <typename Strategy>

[yosin_UTC9, 2017/04/07 04:48:10]

We don't need to use |template <Strategy>|, |computeTextRects()| works only for
DOM tree.

[Hwanseung Lee, 2017/04/07 17:55:53]

Done.

+static Vector<IntRect> computeTextRects(

[yosin_UTC9, 2017/04/07 04:48:10]

It is OK to adding forward declaration before |Range::boundingBox()| with TODO
explains that we'll move impl and remove forward decl of computeTextRects()
here.

[Hwanseung Lee, 2017/04/07 17:55:53]

Done.

+    const EphemeralRangeTemplate<Strategy>& range) {

[yosin_UTC9, 2017/04/07 04:48:10]

nit: const EphemeralRange&

[Hwanseung Lee, 2017/04/07 17:55:53]

Done.

+  const PositionTemplate<Strategy> startPosition = range.startPosition();

[yosin_UTC9, 2017/04/07 04:48:10]

nit const Position&

[Hwanseung Lee, 2017/04/07 17:55:53]

Done.

+  const PositionTemplate<Strategy> endPosition = range.endPosition();
+  Node* startContainer = startPosition.computeContainerNode();
+  DCHECK(startContainer);
+  Node* endContainer = endPosition.computeContainerNode();
+  DCHECK(endContainer);
+  Vector<IntRect> rects;
+  for (Node& node : range.nodes()) {
+    LayoutObject* layoutObject = node.layoutObject();
+    if (!layoutObject || !layoutObject->isText())
+      continue;
+    LayoutText* layoutText = toLayoutText(layoutObject);
+    unsigned startOffset =
+        node == startContainer ? startPosition.offsetInContainerNode() : 0;
+    unsigned endOffset = node == endContainer
+                             ? endPosition.offsetInContainerNode()
+                             : std::numeric_limits<unsigned>::max();
+    layoutText->absoluteRectsForRange(rects, startOffset, endOffset);
+  }
+  return rects;
+}
+
+IntRect Range::computeBoundingBox(const EphemeralRange& range) const {
   IntRect result;
-  Vector<IntRect> rects;

[yosin_UTC9, 2017/04/07 04:48:10]

Just change these two lines:

const Vector<IntRect> rects = computeTextRects(EphemeralRange(this));

-  textRects(rects);
-  for (const IntRect& rect : rects)
+  for (const IntRect& rect : computeTextRects(range))
     result.unite(rect);
   return result;
 }
 
-void Range::textRects(Vector<IntRect>& rects, bool useSelectionHeight) const {
-  Node* startContainer = m_start.container();
-  DCHECK(startContainer);
-  Node* endContainer = m_end.container();
-  DCHECK(endContainer);
-
-  Node* stopNode = pastLastNode();
-  for (Node* node = firstNode(); node != stopNode;
-       node = NodeTraversal::next(*node)) {
-    LayoutObject* r = node->layoutObject();
-    if (!r || !r->isText())
-      continue;
-    LayoutText* layoutText = toLayoutText(r);
-    unsigned startOffset = node == startContainer ? m_start.offset() : 0;
-    unsigned endOffset = node == endContainer
-                             ? m_end.offset()
-                             : std::numeric_limits<unsigned>::max();
-    layoutText->absoluteRectsForRange(rects, startOffset, endOffset,
-                                      useSelectionHeight);
-  }
-}
-
 void Range::textQuads(Vector<FloatQuad>& quads, bool useSelectionHeight) const {
   Node* startContainer = m_start.container();
   DCHECK(startContainer);
   Node* endContainer = m_end.container();
   DCHECK(endContainer);
 
   Node* stopNode = pastLastNode();
   for (Node* node = firstNode(); node != stopNode;
        node = NodeTraversal::next(*node)) {
     LayoutObject* r = node->layoutObject();
@@ skipping 326 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