Make RenderedRectsForMarkers() to ignore disconnected nodes.

third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp

Index: third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
diff --git a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
index 34506d1809171ca1ef41fdc04d995e252f4aa773..732f9746ef9cdbf6c61b57f605a3823cfe4608db 100644
--- a/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
+++ b/third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp
@@ -213,7 +213,7 @@ static void updateMarkerRenderedRect(const Node& node,
     range->setEnd(&const_cast<Node&>(node), marker.endOffset(),
                   IGNORE_EXCEPTION_FOR_TESTING);
   }
-  if (!exceptionState.hadException()) {
+  if (!exceptionState.hadException() && range->isConnected()) {

[yosin_UTC9, 2017/04/04 01:20:39]

This change should not be in this patch.
If we need to satisfy this condition, we should check it call sites,
|node.isConnection()|, actually
there is only one call site
|DocumentMarkerController::renderedRectsForMarkers()|, and should have
|DCEHCK(node.isConnected()| in this function.

Example:
static void updateMarkerRenderedRect(const Node& node, ...) {
  DCHECK(node.isConnected()) << node;
  ...

Vector<IntRect> DocumentMarkerController::renderedRectsForMarkers(...) {
   ...
  for (MarkerMap::iterator nodeIterator = m_markers.begin();
       nodeIterator != end; ++nodeIterator) {
    // inner loop; process each marker in this node
    const Node& node = *nodeIterator->key;
    if (!node.isConnected())
      continue;
  ...
}

[Hwanseung Lee, 2017/04/05 15:21:30]

Done.

     // TODO(yosin): Once we have a |EphemeralRange| version of |boundingBox()|,
     // we should use it instead of |Range| version.
     marker.setRenderedRect(LayoutRect(range->boundingBox()));