Make VisiblePosition::FirstPositionInNode() to take const Node& instead of Node*

third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp

 /*
  * Copyright (C) 2006, 2010 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 366 matching lines @@
           CreateVisiblePosition(current_selection.EndPosition())
               .DeepEquivalent();
 
       HTMLElement* new_list = CreateHTMLElement(GetDocument(), list_tag);
       InsertNodeBefore(new_list, list_element, editing_state);
       if (editing_state->IsAborted())
         return false;
 
       GetDocument().UpdateStyleAndLayoutIgnorePendingStylesheets();
       Node* first_child_in_list =
-          EnclosingListChild(VisiblePosition::FirstPositionInNode(list_element)
+          EnclosingListChild(VisiblePosition::FirstPositionInNode(*list_element)
                                  .DeepEquivalent()
                                  .AnchorNode(),
                              list_element);
       Element* outer_block =
           first_child_in_list && IsBlockFlowElement(*first_child_in_list)
               ? ToElement(first_child_in_list)
               : list_element;
 
       MoveParagraphWithClones(
-          VisiblePosition::FirstPositionInNode(list_element),
+          VisiblePosition::FirstPositionInNode(*list_element),
           VisiblePosition::LastPositionInNode(list_element), new_list,
           outer_block, editing_state);
       if (editing_state->IsAborted())
         return false;
 
       // Manually remove listNode because moveParagraphWithClones sometimes
       // leaves it behind in the document. See the bug 33668 and
       // editing/execCommand/insert-list-orphaned-item-with-nested-lists.html.
       // FIXME: This might be a bug in moveParagraphWithClones or
       // deleteSelection.
@@ skipping 45 matching lines @@
     EditingState* editing_state) {
   // Since, unlistify paragraph inserts nodes into parent and removes node
   // from parent, if parent of |listElement| should be editable.
   DCHECK(HasEditableStyle(*list_element->parentNode()));
   Node* next_list_child;
   Node* previous_list_child;
   VisiblePosition start;
   VisiblePosition end;
   DCHECK(list_child_node);
   if (isHTMLLIElement(*list_child_node)) {
-    start = VisiblePosition::FirstPositionInNode(list_child_node);
+    start = VisiblePosition::FirstPositionInNode(*list_child_node);
     end = VisiblePosition::LastPositionInNode(list_child_node);
     next_list_child = list_child_node->nextSibling();
     previous_list_child = list_child_node->previousSibling();
   } else {
     // A paragraph is visually a list item minus a list marker.  The paragraph
     // will be moved.
     start = StartOfParagraph(original_start, kCanSkipOverEditingBoundary);
     end = EndOfParagraph(start, kCanSkipOverEditingBoundary);
     next_list_child = EnclosingListChild(
         NextPositionOf(end).DeepEquivalent().AnchorNode(), list_element);
@@ skipping 209 matching lines @@
       EndOfParagraph(valid_pos, kCanSkipOverEditingBoundary);
   MoveParagraph(start, end, VisiblePosition::BeforeNode(*placeholder),
                 editing_state, kPreserveSelection);
 }
 
 DEFINE_TRACE(InsertListCommand) {
   CompositeEditCommand::Trace(visitor);
 }
 
 }  // namespace blink