Make RenderedRectsForMarkers() to ignore disconnected nodes.

third_party/WebKit/Source/core/editing/EphemeralRange.h

Index: third_party/WebKit/Source/core/editing/EphemeralRange.h
diff --git a/third_party/WebKit/Source/core/editing/EphemeralRange.h b/third_party/WebKit/Source/core/editing/EphemeralRange.h
index acdbafb3ef6793e8f15208cd919e0af7b5ab583f..c868559d62a167f3156271f1bb3f72d4b3df9962 100644
--- a/third_party/WebKit/Source/core/editing/EphemeralRange.h
+++ b/third_party/WebKit/Source/core/editing/EphemeralRange.h
@@ -105,6 +105,9 @@ class CORE_TEMPLATE_CLASS_EXPORT EphemeralRangeTemplate final {
   PositionTemplate<Strategy> startPosition() const;
   PositionTemplate<Strategy> endPosition() const;
 
+  void textRects(Vector<IntRect>&, bool useSelectionHeight = false) const;

[Xiaocheng, 2017/03/29 19:51:07]

This function is just implementation detail and shouldn't appear in a header
file. Besides, |userSelectionHeight| is never used.

+  IntRect boundingBox() const;

[Xiaocheng, 2017/03/29 19:51:07]

According to the discussion in https://codereview.chromium.org/2775663008, we
should not make EphemeralRange depend on layout.

Could you introduce new functions in VisibleUnits:

IntRect boundingBox(const EphemeralRange&);
IntRect boundingBox(const EphemeralRangeInFlatTree&);

+
   Node* commonAncestorContainer() const;
 
   // Returns true if |m_startPositoin| == |m_endPosition| or |isNull()|.