Make Position::AfterNode() to take const Node& instead of Node*

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 1223 matching lines @@
   const char* body_content =
       "<p id='host'><b id='one'>1</b></p><b id='two'>22</b>";
   const char* shadow_content =
       "<b id='two'>22</b><content select=#one></content><b id='three'>333</b>";
   SetBodyContent(body_content);
   SetShadowContent(shadow_content, "host");
 
   Element* host = GetDocument().getElementById("host");
 
   EXPECT_EQ(Position::LastPositionInNode(host),
-            MostForwardCaretPosition(Position::AfterNode(host)));
+            MostForwardCaretPosition(Position::AfterNode(*host)));
   EXPECT_EQ(PositionInFlatTree::LastPositionInNode(host),
-            MostForwardCaretPosition(PositionInFlatTree::AfterNode(host)));
+            MostForwardCaretPosition(PositionInFlatTree::AfterNode(*host)));
 }
 
 TEST_F(VisibleUnitsTest, mostBackwardCaretPositionFirstLetter) {
   // Note: first-letter pseudo element contains letter and punctuations.
   const char* body_content =
       "<style>p:first-letter {color:red;}</style><p id=sample> (2)45 </p>";
   SetBodyContent(body_content);
 
   Node* sample = GetDocument().getElementById("sample")->firstChild();
 
@@ skipping 11 matching lines @@
             MostBackwardCaretPosition(Position(sample, 5)));
   EXPECT_EQ(Position(sample, 6),
             MostBackwardCaretPosition(Position(sample, 6)));
   EXPECT_EQ(Position(sample, 6),
             MostBackwardCaretPosition(Position(sample, 7)));
   EXPECT_EQ(Position(sample, 6),
             MostBackwardCaretPosition(
                 Position::LastPositionInNode(sample->parentNode())));
   EXPECT_EQ(
       Position(sample, 6),
-      MostBackwardCaretPosition(Position::AfterNode(sample->parentNode())));
+      MostBackwardCaretPosition(Position::AfterNode(*sample->parentNode())));
   EXPECT_EQ(Position::LastPositionInNode(GetDocument().body()),
             MostBackwardCaretPosition(
                 Position::LastPositionInNode(GetDocument().body())));
 }
 
 TEST_F(VisibleUnitsTest, mostBackwardCaretPositionFirstLetterSplit) {
   V8TestingScope scope;
 
   const char* body_content =
       "<style>p:first-letter {color:red;}</style><p id=sample>abc</p>";
@@ skipping 11 matching lines @@
             MostBackwardCaretPosition(Position(first_letter, 1)));
   EXPECT_EQ(Position(first_letter, 1),
             MostBackwardCaretPosition(Position(remaining, 0)));
   EXPECT_EQ(Position(remaining, 1),
             MostBackwardCaretPosition(Position(remaining, 1)));
   EXPECT_EQ(Position(remaining, 2),
             MostBackwardCaretPosition(Position(remaining, 2)));
   EXPECT_EQ(Position(remaining, 2),
             MostBackwardCaretPosition(Position::LastPositionInNode(sample)));
   EXPECT_EQ(Position(remaining, 2),
-            MostBackwardCaretPosition(Position::AfterNode(sample)));
+            MostBackwardCaretPosition(Position::AfterNode(*sample)));
 }
 
 TEST_F(VisibleUnitsTest, mostForwardCaretPositionAfterAnchor) {
   const char* body_content = "<p id='host'><b id='one'>1</b></p>";
   const char* shadow_content =
       "<b id='two'>22</b><content select=#one></content><b id='three'>333</b>";
   SetBodyContent(body_content);
   ShadowRoot* shadow_root = SetShadowContent(shadow_content, "host");
   UpdateAllLifecyclePhases();
 
   Element* host = GetDocument().getElementById("host");
   Element* one = GetDocument().getElementById("one");
   Element* three = shadow_root->getElementById("three");
 
   EXPECT_EQ(Position(one->firstChild(), 1),
-            MostBackwardCaretPosition(Position::AfterNode(host)));
+            MostBackwardCaretPosition(Position::AfterNode(*host)));
   EXPECT_EQ(PositionInFlatTree(three->firstChild(), 3),
-            MostBackwardCaretPosition(PositionInFlatTree::AfterNode(host)));
+            MostBackwardCaretPosition(PositionInFlatTree::AfterNode(*host)));
 }
 
 TEST_F(VisibleUnitsTest, mostForwardCaretPositionFirstLetter) {
   // Note: first-letter pseudo element contains letter and punctuations.
   const char* body_content =
       "<style>p:first-letter {color:red;}</style><p id=sample> (2)45 </p>";
   SetBodyContent(body_content);
 
   Node* sample = GetDocument().getElementById("sample")->firstChild();
 
@@ skipping 175 matching lines @@
       PreviousPositionOf(CreateVisiblePosition(position)).DeepEquivalent());
 }
 
 TEST_F(VisibleUnitsTest, rendersInDifferentPositionAfterAnchor) {
   const char* body_content = "<p id='sample'>00</p>";
   SetBodyContent(body_content);
   Element* sample = GetDocument().getElementById("sample");
 
   EXPECT_FALSE(RendersInDifferentPosition(Position(), Position()));
   EXPECT_FALSE(
-      RendersInDifferentPosition(Position(), Position::AfterNode(sample)))
+      RendersInDifferentPosition(Position(), Position::AfterNode(*sample)))
       << "if one of position is null, the reuslt is false.";
-  EXPECT_FALSE(RendersInDifferentPosition(Position::AfterNode(sample),
+  EXPECT_FALSE(RendersInDifferentPosition(Position::AfterNode(*sample),
                                           Position(sample, 1)));
   EXPECT_FALSE(RendersInDifferentPosition(Position::LastPositionInNode(sample),
                                           Position(sample, 1)));
 }
 
 TEST_F(VisibleUnitsTest, rendersInDifferentPositionAfterAnchorWithHidden) {
   const char* body_content =
       "<p><span id=one>11</span><span id=two style='display:none'>  "
       "</span></p>";
   SetBodyContent(body_content);
@@ skipping 22 matching lines @@
 
 TEST_F(VisibleUnitsTest, renderedOffset) {
   const char* body_content =
       "<div contenteditable><span id='sample1'>1</span><span "
       "id='sample2'>22</span></div>";
   SetBodyContent(body_content);
   Element* sample1 = GetDocument().getElementById("sample1");
   Element* sample2 = GetDocument().getElementById("sample2");
 
   EXPECT_FALSE(
-      RendersInDifferentPosition(Position::AfterNode(sample1->firstChild()),
+      RendersInDifferentPosition(Position::AfterNode(*sample1->firstChild()),
                                  Position(sample2->firstChild(), 0)));
   EXPECT_FALSE(RendersInDifferentPosition(
       Position::LastPositionInNode(sample1->firstChild()),
       Position(sample2->firstChild(), 0)));
 }
 
 TEST_F(VisibleUnitsTest, startOfDocument) {
   const char* body_content = "<a id=host><b id=one>1</b><b id=two>22</b></a>";
   const char* shadow_content =
       "<p><content select=#two></content></p><p><content "
@@ skipping 339 matching lines @@
   Element* const one = GetDocument().getElementById("one");
   Element* const two = GetDocument().getElementById("two");
   const VisiblePosition& visible_position =
       CreateVisiblePosition(Position::LastPositionInNode(two));
   EXPECT_EQ(Position(one->firstChild(), 7),
             PreviousRootInlineBoxCandidatePosition(
                 two->lastChild(), visible_position, kContentIsEditable));
 }
 
 }  // namespace blink