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

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
  * reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 198 matching lines @@
     start_box = FindRightNonPseudoNodeInlineBox(*root_box);
     if (!start_box)
       return PositionWithAffinityTemplate<Strategy>();
     start_node = start_box->GetLineLayoutItem().NonPseudoNode();
   }
 
   return PositionWithAffinityTemplate<Strategy>(
       start_node->IsTextNode()
           ? PositionTemplate<Strategy>(ToText(start_node),
                                        ToInlineTextBox(start_box)->Start())
-          : PositionTemplate<Strategy>::BeforeNode(start_node));
+          : PositionTemplate<Strategy>::BeforeNode(*start_node));
 }
 
 template <typename Strategy>
 PositionWithAffinityTemplate<Strategy> StartOfLineAlgorithm(
     const PositionWithAffinityTemplate<Strategy>& c) {
   // TODO: this is the current behavior that might need to be fixed.
   // Please refer to https://bugs.webkit.org/show_bug.cgi?id=49107 for detail.
   PositionWithAffinityTemplate<Strategy> vis_pos =
       StartPositionForLine(c, kUseInlineBoxOrdering);
   return HonorEditingBoundaryAtOrBefore(vis_pos, c.GetPosition());
@@ skipping 64 matching lines @@
       HighestEditableRoot(visible_position.DeepEquivalent(), editable_type);
   Node* const previous_node =
       FindNodeInPreviousLine(*node, visible_position, editable_type);
   for (Node* runner = previous_node; runner && !runner->IsShadowRoot();
        runner = PreviousLeafWithSameEditability(runner, editable_type)) {
     if (HighestEditableRootOfNode(*runner, editable_type) != highest_root)
       break;
 
     const Position& candidate =
         isHTMLBRElement(*runner)
-            ? Position::BeforeNode(runner)
+            ? Position::BeforeNode(*runner)
             : Position::EditingPositionOf(runner, CaretMaxOffset(runner));
     if (IsVisuallyEquivalentCandidate(candidate))
       return candidate;
   }
   return Position();
 }
 
 Position NextRootInlineBoxCandidatePosition(
     Node* node,
     const VisiblePosition& visible_position,
@@ skipping 107 matching lines @@
     // TODO(editing-dev): We should consider text-direction of line to
     // find non-pseudo node.
     end_box = FindLeftNonPseudoNodeInlineBox(*root_box);
     if (!end_box)
       return VisiblePositionTemplate<Strategy>();
     end_node = end_box->GetLineLayoutItem().NonPseudoNode();
   }
 
   if (isHTMLBRElement(*end_node)) {
     return CreateVisiblePosition(
-        PositionTemplate<Strategy>::BeforeNode(end_node),
+        PositionTemplate<Strategy>::BeforeNode(*end_node),
         VP_UPSTREAM_IF_POSSIBLE);
   }
   if (end_box->IsInlineTextBox() && end_node->IsTextNode()) {
     InlineTextBox* end_text_box = ToInlineTextBox(end_box);
     int end_offset = end_text_box->Start();
     if (!end_text_box->IsLineBreak())
       end_offset += end_text_box->Len();
     return CreateVisiblePosition(
         PositionTemplate<Strategy>(ToText(end_node), end_offset),
         VP_UPSTREAM_IF_POSSIBLE);
@@ skipping 304 matching lines @@
   // to the end of the line we're on.
   Element* root_element = HasEditableStyle(*node, editable_type)
                               ? RootEditableElement(*node, editable_type)
                               : node->GetDocument().documentElement();
   if (!root_element)
     return VisiblePosition();
   return VisiblePosition::LastPositionInNode(root_element);
 }
 
 }  // namespace blink