createFragmentFromMarkupWithContext() should use EphemeralRange.

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 130 matching lines @@
   // Tree iterators have only |operator !=| ATM.
   EXPECT_FALSE(iterable.begin() != iterable.end());
 
   const EphemeralRange singlePositionRange(getBodyRange()->startPosition());
   EXPECT_FALSE(singlePositionRange.isNull());
   EXPECT_EQ(std::string(), traverseRange(singlePositionRange));
   EXPECT_EQ(singlePositionRange.startPosition().nodeAsRangeFirstNode(),
             singlePositionRange.endPosition().nodeAsRangePastLastNode());
 }
 
+TEST_F(EphemeralRangeTest, commonAncesstorDOM) {
+  const char* bodyContent =
+      "<p id='host'>00<b id='one'>11</b><b id='two'>22</b><b "
+      "id='three'>33</b></p>";
+  setBodyContent(bodyContent);
+
+  Range* range = getBodyRange();

[Xiaocheng, 2017/03/05 06:31:19]

There's no need to create an EphemeralRange from Range. Just do this:

const Position startPosition(document().getElementById("one"), 0);
const Position endPosition(document().getElementById("two"), 0);
const EphemeralRange range(startPosition, endPosition);
...

[tanvir, 2017/03/06 07:05:11]

Done.

+  range->setStart(document().getElementById("one"), 0,
+                  IGNORE_EXCEPTION_FOR_TESTING);
+  range->setEnd(document().getElementById("two"), 0,
+                IGNORE_EXCEPTION_FOR_TESTING);
+
+  const EphemeralRange ephemeralRange(range->startPosition(),
+                                      range->endPosition());
+  const Node* commonAncestor = document().getElementById("host");
+  EXPECT_EQ(commonAncestor, ephemeralRange.commonAncestorContainer());
+}
+
+TEST_F(EphemeralRangeTest, commonAncesstorFlatTree) {
+  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);
+  setShadowContent(shadowContent, "host");

[Xiaocheng, 2017/03/05 06:31:19]

Do instead:

...
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());

Note that |document().getElementById("five")| returns nullptr, because node
#five is hidden in the shadow tree.

[tanvir, 2017/03/06 07:05:11]

Done the changes!! Thanks for the explanation.

+
+  Range* range = getBodyRange();
+  range->setStart(document().getElementById("one"), 0,
+                  IGNORE_EXCEPTION_FOR_TESTING);
+  range->setEnd(document().getElementById("five"), 0,
+                IGNORE_EXCEPTION_FOR_TESTING);
+
+  const EphemeralRangeInFlatTree ephemeralRange(
+      toPositionInFlatTree(range->startPosition()),
+      toPositionInFlatTree(range->endPosition()));
+  EXPECT_EQ(document().documentElement(),
+            ephemeralRange.commonAncestorContainer());
+}
+
 }  // namespace blink