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

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 231 matching lines @@
 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();
   EXPECT_EQ(SelectionInDOMTree::Builder(result_in_dom_tree)
-                .Collapse(Position::BeforeNode(input))
+                .Collapse(Position::BeforeNode(*input))
                 .Extend(Position(last_child, 3))
                 .Build(),
             result_in_dom_tree);
   EXPECT_EQ(SelectionInFlatTree::Builder(result_in_flat_tree)
-                .Collapse(PositionInFlatTree::BeforeNode(input))
+                .Collapse(PositionInFlatTree::BeforeNode(*input))
                 .Extend(PositionInFlatTree(last_child, 3))
                 .Build(),
             result_in_flat_tree);
 }
 
 TEST_F(FrameSelectionTest, SelectAllWithUnselectableRoot) {
   Element* select = GetDocument().createElement("select");
   GetDocument().ReplaceChild(select, GetDocument().documentElement());
   GetDocument().UpdateStyleAndLayout();
   Selection().SelectAll();
@@ skipping 698 matching lines @@
 // crbug.com/725457
 TEST_F(FrameSelectionTest, InconsistentVisibleSelectionNoCrash) {
   SetBodyContent("foo<div id=host><span id=anchor>bar</span></div>baz");
   SetShadowContent("shadow", "host");
 
   Element* anchor = GetDocument().getElementById("anchor");
 
   // |start| and |end| are valid Positions in DOM tree, but do not participate
   // in flat tree. They should be canonicalized to null VisiblePositions, but
   // are currently not. See crbug.com/729636 for details.
-  const Position& start = Position::BeforeNode(anchor);
+  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