Make FrameSelection to hold non-canonicalized positions

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

Index: third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp b/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
index e5e13b0e99c469b8e4185a2418c124bb3eaaf2aa..a0749aea8499222f4b201071312f0ea85302318f 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelectionTest.cpp
@@ -266,17 +266,14 @@ TEST_F(FrameSelectionTest, updateIfNeededAndFrameCaret) {
   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())

[tkent, 2017/02/10 08:47:41]

Why isn't it Position(HTML,0)?

[yosin_UTC9, 2017/02/10 10:13:20]

My reasoning was wrong. I forgot to consider HEAD element.

Positoin() is correct since 
  VisiblePosition(Postion(After, HEAD)) == Position() and
  VisiblePosition(Postion(HTML, 1)) == Position().
// I think this is an issue of VP canonicalization, VP(After, HEAD) should be
VP(After, HEAD)


as L275's EXPECT_EQ(Position(), selection().start())

DOM tree before removal is:
#document
  HTML
    HEAD (editable)
    BODY (editable)
      STYLE id="sample" (editable)

DOM tree after removal:
#document (editable)
  HTML (editable)
    HEAD (editable)

Before removal: selection is STYLE,0
After removal: selection is After HEAD; this is identical to HTML,1

+      << "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());
 }