Make RenderedRectsForMarkers() to ignore disconnected nodes.

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

 /*
  * (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 Apple Inc. All rights
  * reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 12 matching lines @@
  *
  */
 
 #ifndef Range_h
 #define Range_h
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "core/CoreExport.h"
 #include "core/dom/RangeBoundaryPoint.h"
+#include "core/editing/EphemeralRange.h"
 #include "platform/geometry/FloatRect.h"
 #include "platform/geometry/IntRect.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Forward.h"
 
 namespace blink {
 
 class ClientRect;
 class ClientRectList;
 class ContainerNode;
@@ skipping 90 matching lines @@
 
   const Position startPosition() const { return m_start.toPosition(); }
   const Position endPosition() const { return m_end.toPosition(); }
   void setStart(const Position&, ExceptionState& = ASSERT_NO_EXCEPTION);
   void setEnd(const Position&, ExceptionState& = ASSERT_NO_EXCEPTION);
 
   Node* firstNode() const;
   Node* pastLastNode() const;
 
   // Not transform-friendly
-  void textRects(Vector<IntRect>&, bool useSelectionHeight = false) const;
   IntRect boundingBox() const;
+  IntRect computeBoundingBox(const EphemeralRange&) const;

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

No need to introduce |computeBondingBox()| in this patch.
This patch is just replace |textRects| to |computeTextRects(const
EphemeralRange&)|.

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

Done.

 
   // Transform-friendly
   void textQuads(Vector<FloatQuad>&, bool useSelectionHeight = false) const;
   void getBorderAndTextQuads(Vector<FloatQuad>&) const;
   FloatRect boundingRect() const;
 
   void nodeChildrenWillBeRemoved(ContainerNode&);
   void nodeWillBeRemoved(Node&);
 
   void didInsertText(Node*, unsigned offset, unsigned length);
@@ skipping 66 matching lines @@
 using RangeVector = HeapVector<Member<Range>>;
 
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const blink::Range*);
 #endif
 
 #endif  // Range_h