Make RenderedRectsForMarkers() to ignore disconnected nodes.

third_party/WebKit/Source/core/editing/EphemeralRangeTest.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/editing/EphemeralRange.h"
 
 #include "core/dom/Range.h"
 #include "core/editing/EditingTestBase.h"
 #include <sstream>
 
@@ skipping 203 matching lines @@
       "<p id='five'>55555</p>";
   setBodyContent(bodyContent);
   ShadowRoot* shadowRoot = setShadowContent(shadowContent, "host");
 
   const PositionInFlatTree startPosition(document().getElementById("one"), 0);
   const PositionInFlatTree endPosition(shadowRoot->getElementById("five"), 0);
   const EphemeralRangeInFlatTree range(startPosition, endPosition);
   EXPECT_EQ(document().getElementById("host"), range.commonAncestorContainer());
 }
 
+TEST_F(EphemeralRangeTest, boundingBox) {

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

Please move the test case to VisibleUnitsTest, and introduce another test case
for EphemeralRangeInFlatTree.

+  const char* bodyContent =
+      "<p id='host'>1"
+      "<b id='one'>22</b>"
+      "<b id='two'>333</b>"
+      "<b id='three'>4444</b>"
+      "</p>";
+  setBodyContent(bodyContent);
+
+  const Position startPosition(document().getElementById("one"), 0);
+  const Position endPosition(document().getElementById("three"), 0);
+  Range* range = getBodyRange();
+  range->setStart(document().getElementById("one"), 0,
+                  IGNORE_EXCEPTION_FOR_TESTING);
+  range->setEnd(document().getElementById("three"), 0,
+                IGNORE_EXCEPTION_FOR_TESTING);
+  const EphemeralRange ephemeralRange(startPosition, endPosition);
+  EXPECT_EQ(range->boundingBox().toString(),

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

Please inspect the return value directly. No need to compare it to
Range::boundingBox.

+            ephemeralRange.boundingBox().toString());
+}
+
 }  // namespace blink