Make FrameSelection::MoveRangeSelection() not to set live granularity selection

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

 // Copyright 2014 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/FrameSelection.h"
 
 #include <memory>
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
@@ skipping 220 matching lines @@
                                  CreateVisiblePosition(Position(text, 2)),
                                  kCharacterGranularity);
   EXPECT_EQ_SELECTED_TEXT("o B");
   // "Fo<o B|ar Baz," with the Word granularity.
   Selection().MoveRangeSelection(CreateVisiblePosition(Position(text, 5)),
                                  CreateVisiblePosition(Position(text, 2)),
                                  kWordGranularity);
   EXPECT_EQ_SELECTED_TEXT("Foo Bar");
 }
 
+TEST_F(FrameSelectionTest, MoveRangeSelectionNoLiveness) {
+  SetBodyContent("<span id=sample>xyz</span>");
+  Element* const sample = GetDocument().getElementById("sample");
+  // Select as: <span id=sample>^xyz|</span>
+  Selection().MoveRangeSelection(
+      CreateVisiblePosition(Position(sample->firstChild(), 1)),
+      CreateVisiblePosition(Position(sample->firstChild(), 1)),
+      kWordGranularity);

[yoichio, 2017/07/05 08:57:24]

Could you expect_eq(SelectText(), 'xyz')?

[yosin_UTC9, 2017/07/05 09:15:49]

Done.

+  sample->insertBefore(Text::Create(GetDocument(), "abc"),
+                       sample->firstChild());
+  GetDocument().UpdateStyleAndLayout();
+  const VisibleSelection& selection =
+      Selection().ComputeVisibleSelectionInDOMTree();
+  // Inserting "abc" before "xyz" should not affect to selection.
+  EXPECT_EQ(Position(sample->lastChild(), 0), selection.Start());
+  EXPECT_EQ(Position(sample->lastChild(), 3), selection.End());

[yoichio, 2017/07/05 08:57:24]

Could you expect_eq(SelectText(), 'xyz') and  expect_eq(sample->innerText(),
'abcxyz') ?

[yosin_UTC9, 2017/07/05 09:15:49]

Done.

+}
+
 // For http://crbug.com/695317
 TEST_F(FrameSelectionTest, SelectAllWithInputElement) {
   SetBodyContent("<input>123");
   Element* const input = GetDocument().QuerySelector("input");
   Node* const last_child = GetDocument().body()->lastChild();
   Selection().SelectAll();
   const SelectionInDOMTree& result_in_dom_tree =
       Selection().ComputeVisibleSelectionInDOMTree().AsSelection();
   const SelectionInFlatTree& result_in_flat_tree =
       Selection().ComputeVisibleSelectionInFlatTree().AsSelection();
@@ skipping 725 matching lines @@
   const Position& start = Position::BeforeNode(*anchor);
   const Position& end = Position::AfterNode(*anchor);
   Selection().SetSelection(
       SelectionInDOMTree::Builder().Collapse(start).Extend(end).Build());
 
   // Shouldn't crash inside.
   EXPECT_FALSE(Selection().SelectionHasFocus());
 }
 
 }  // namespace blink