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

third_party/WebKit/Source/core/editing/VisibleSelectionTest.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/VisibleSelection.h"
 
 #include "core/dom/Range.h"
 #include "core/editing/EditingTestBase.h"
 #include "core/editing/SelectionAdjuster.h"
 
@@ skipping 66 matching lines @@
       "<input type=checkbox>"
       "<div style='user-select:none'>abc</div>");
   Element* const input = GetDocument().QuerySelector("input");
 
   // Simulate double-clicking "abc".
   // TODO(editing-dev): We should remove above comment once we fix [1].
   // [1] http://crbug.com/701657 double-click on user-select:none should not
   // compute selection.
   VisibleSelection selection =
       CreateVisibleSelection(SelectionInDOMTree::Builder()
-                                 .Collapse(Position::BeforeNode(input))
+                                 .Collapse(Position::BeforeNode(*input))
                                  .Extend(Position::AfterNode(input))
                                  .SetGranularity(kWordGranularity)
                                  .Build());
   selection.AppendTrailingWhitespace();
 
-  EXPECT_EQ(Position::BeforeNode(input), selection.Start());
+  EXPECT_EQ(Position::BeforeNode(*input), selection.Start());
   EXPECT_EQ(Position::AfterNode(input), selection.End());
 }
 
 TEST_F(VisibleSelectionTest, expandUsingGranularity) {
   const char* body_content =
       "<span id=host><a id=one>1</a><a id=two>22</a></span>";
   const char* shadow_content =
       "<p><b id=three>333</b><content select=#two></content><b "
       "id=four>4444</b><span id=space>  </span><content "
       "select=#one></content><b id=five>55555</b></p>";
@@ skipping 146 matching lines @@
   Element* const html_element = GetDocument().documentElement();
   Element* const input = GetDocument().QuerySelector("input");
   Node* const last_child = GetDocument().body()->lastChild();
 
   const VisibleSelection& visible_selectin_in_dom_tree = CreateVisibleSelection(
       SelectionInDOMTree::Builder()
           .Collapse(Position::FirstPositionInNode(html_element))
           .Extend(Position::LastPositionInNode(html_element))
           .Build());
   EXPECT_EQ(SelectionInDOMTree::Builder()
-                .Collapse(Position::BeforeNode(input))
+                .Collapse(Position::BeforeNode(*input))
                 .Extend(Position(last_child, 3))
                 .Build(),
             visible_selectin_in_dom_tree.AsSelection());
 
   const VisibleSelectionInFlatTree& visible_selectin_in_flat_tree =
       CreateVisibleSelection(
           SelectionInFlatTree::Builder()
               .Collapse(PositionInFlatTree::FirstPositionInNode(html_element))
               .Extend(PositionInFlatTree::LastPositionInNode(html_element))
               .Build());
   EXPECT_EQ(SelectionInFlatTree::Builder()
-                .Collapse(PositionInFlatTree::BeforeNode(input))
+                .Collapse(PositionInFlatTree::BeforeNode(*input))
                 .Extend(PositionInFlatTree(last_child, 3))
                 .Build(),
             visible_selectin_in_flat_tree.AsSelection());
 }
 
 TEST_F(VisibleSelectionTest, ShadowCrossing) {
   const char* body_content =
       "<p id='host'>00<b id='one'>11</b><b id='two'>22</b>33</p>";
   const char* shadow_content =
       "<a><span id='s4'>44</span><content select=#two></content><span "
@@ skipping 249 matching lines @@
   Element* host = GetDocument().getElementById("host");
   host->AppendChild(sample);
   GetDocument().UpdateStyleAndLayout();
 
   // Simulates to restore selection from undo stack.
   selection = CreateVisibleSelection(selection.AsSelection());
   EXPECT_EQ(Position(sample->firstChild(), 0), selection.Start());
 }
 
 }  // namespace blink