LayoutObject::absoluteBoundingBoxRectForRange() should take EphemeralRange

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

 // Copyright 2015 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/VisibleUnits.h"
 
 #include <ostream>  // NOLINT
 #include "bindings/core/v8/V8BindingForTesting.h"
 #include "core/dom/Text.h"
 #include "core/editing/EditingTestBase.h"
@@ skipping 1959 matching lines @@
 TEST_F(VisibleUnitsTest,
        canonicalizationWithCollapsedSpaceAndIsolatedCombiningCharacter) {
   setBodyContent("<p>  &#x20E3;</p>");  // Leading space is necessary
 
   Node* paragraph = document().querySelector("p");
   Node* text = paragraph->firstChild();
   Position start = canonicalPositionOf(Position::beforeNode(paragraph));
   EXPECT_EQ(Position(text, 2), start);
 }
 
+TEST_F(VisibleUnitsTest, textQuadDOM) {
+  const char* bodyContent =
+      "<p id='host'>00"
+      "<b id='one'>11111</b>"
+      "<b id='two'>22</b>"
+      "<b id='three'>33</b>"
+      "</p>";
+  setBodyContent(bodyContent);
+
+  const Position startPosition(document().getElementById("host"), 0);
+  const Position midPosition(document().getElementById("two"), 0);
+  const Position endPosition(document().getElementById("three"), 0);
+  const EphemeralRange rangeStartToTwo(startPosition, midPosition);
+  const EphemeralRange rangeStartToThree(startPosition, endPosition);
+  Vector<FloatQuad> quadsStartToTwo;
+  Vector<FloatQuad> quadsStartToThree;
+  textQuads(quadsStartToTwo, rangeStartToTwo);
+  textQuads(quadsStartToThree, rangeStartToThree);
+
+  EXPECT_EQ(1u, quadsStartToTwo.size());
+  EXPECT_EQ(1u, quadsStartToThree.size());
+  EXPECT_EQ("10,8; 15,8; 15,9; 10,9", quadsStartToTwo[0].toString());

[Xiaocheng, 2017/03/31 18:01:19]

Wait... Doesn't https://codereview.chromium.org/2763013002 break the behavior?

"10,8; 15,8; 15,9; 10,9" is just the quad for "11111". The quad for "00" is
missing.

[tanvir, 2017/04/03 13:00:48]

Based on the discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=707627, 
a patch is submitted to take care of this.
And so i have moved back to previous TextCase of #Patch7. 

  

+  EXPECT_EQ("15,8; 17,8; 17,9; 15,9", quadsStartToThree[0].toString());

[Xiaocheng, 2017/03/31 18:01:19]

Same here, "15,8; 17,8; 17,9; 15,9" is the quad for "22". The quads for "00" and
"11111" are missing.

+}
+
+TEST_F(VisibleUnitsTest, textQuadFlatTree) {
+  const char* bodyContent =
+      "<b id='zero'>0</b>"
+      "<p id='host'>"
+      "<b id='one'>1</b>"
+      "<b id='two'>22</b>"
+      "</p>"
+      "<b id='three'>333</b>";
+  const char* shadowContent =
+      "<p id='four'>4444</p>"
+      "<content select=#two></content>"
+      "<content select=#one></content>"
+      "<p id='five'>55555</p>";
+  setBodyContent(bodyContent);
+  ShadowRoot* shadowRoot = setShadowContent(shadowContent, "host");
+
+  const PositionInFlatTree startPosition(document().getElementById("host"), 0);
+  const PositionInFlatTree threePosition(document().getElementById("three"), 0);
+  const PositionInFlatTree fivePosition(shadowRoot->getElementById("five"), 0);
+  const EphemeralRangeInFlatTree rangeStartToThree(startPosition,
+                                                   threePosition);
+  const EphemeralRangeInFlatTree rangeStartToFive(startPosition, fivePosition);
+  Vector<FloatQuad> quadsStartToThree;
+  Vector<FloatQuad> quadsStartToFive;
+  textQuads(quadsStartToThree, rangeStartToThree);
+  textQuads(quadsStartToFive, rangeStartToFive);
+
+  EXPECT_EQ(1u, quadsStartToThree.size());
+  EXPECT_EQ(1u, quadsStartToFive.size());
+  EXPECT_EQ("8,14; 13,14; 13,15; 8,15", quadsStartToThree[0].toString());
+  EXPECT_EQ("10,12; 11,12; 11,13; 10,13", quadsStartToFive[0].toString());
+}
 }  // namespace blink