Make RenderedRectsForMarkers() to ignore disconnected nodes.

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 4030 matching lines @@
 }
 
 Position skipWhitespace(const Position& position) {
   return skipWhitespaceAlgorithm(position);
 }
 
 PositionInFlatTree skipWhitespace(const PositionInFlatTree& position) {
   return skipWhitespaceAlgorithm(position);
 }
 
+template <typename Strategy>
+Vector<IntRect> computeTextRects(

[yosin_UTC9, 2017/04/06 04:26:04]

In this patch, please do this change in "Range.cpp" rather than moving to
"VisibleUnits.cpp" for
ease of tracking change of Range to EphemeralRange.

In other words, this patch should be change Range.cpp only. Following patches
will
 1. Move computeTextRects() to "VisibleUnits.cpp"
 2. Change call sites to use computeTextRects()

Also, we don't expect behavior changes by this effort, since this is just
re-factoring.

+    const EphemeralRangeTemplate<Strategy>& range) {
+  const PositionTemplate<Strategy> startPosition = range.startPosition();
+  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 computeBoundingBox(const EphemeralRange& range) {
+  IntRect result;
+  for (const IntRect& rect : computeTextRects(range))
+    result.unite(rect);
+  return result;
+}
+
+IntRect computeBoundingBox(const EphemeralRangeInFlatTree& range) {
+  IntRect result;
+  for (const IntRect& rect : computeTextRects(range))
+    result.unite(rect);
+  return result;
+}
+
 }  // namespace blink