Make FrameSelection to hold non-canonicalized positions

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 248 matching lines @@
 TEST_F(FrameSelectionTest, updateIfNeededAndFrameCaret) {
   setBodyContent("<style id=sample></style>");
   document().setDesignMode("on");
   updateAllLifecyclePhases();
   Element* sample = document().getElementById("sample");
   selection().setSelection(
       SelectionInDOMTree::Builder().collapse(Position(sample, 0)).build());
   EXPECT_EQ(Position(document().body(), 0), selection().start());
   EXPECT_EQ(selection().start(), caretPosition().position());
   document().body()->remove();
-  // TODO(yosin): Once lazy canonicalization implemented, selection.start
-  // should be Position(HTML, 0).
-  EXPECT_EQ(Position(document().documentElement(), 1), selection().start());
+  EXPECT_EQ(Position(), selection().start())
+      << "Selection has been removed by BODY.remove().";
   EXPECT_EQ(selection().start(), caretPosition().position());
   document().updateStyleAndLayout();
   selection().updateIfNeeded();
 
-  // TODO(yosin): Once lazy canonicalization implemented, selection.start
-  // should be Position(HTML, 0).
   EXPECT_EQ(Position(), selection().start())
-      << "updateIfNeeded() makes selection to null.";
+      << "selection().updateIfNeeded() does nothing.";
   EXPECT_EQ(selection().start(), caretPosition().position());
 }
 
 }  // namespace blink