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

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

 /*
  * Copyright (C) 2005, 2006, 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1050 matching lines @@
 
   for (Node* node = run_start; node && node != past_end_node;
        node = NodeTraversal::Next(*node)) {
     if (node->hasChildren())
       continue;
     // We don't consider m_isInlineElementToRemoveFunction here because we never
     // apply style when m_isInlineElementToRemoveFunction is specified
     if (!style->StyleIsPresentInComputedStyleOfNode(node))
       return true;
     if (styled_inline_element_ &&
-        !EnclosingElementWithTag(Position::BeforeNode(node),
+        !EnclosingElementWithTag(Position::BeforeNode(*node),
                                  styled_inline_element_->TagQName()))
       return true;
   }
   return false;
 }
 
 void ApplyStyleCommand::RemoveConflictingInlineStyleFromRun(
     EditingStyle* style,
     Member<Node>& run_start,
     Member<Node>& run_end,
@@ skipping 501 matching lines @@
   Position new_end;
   if (start.ComputeContainerNode() == end.ComputeContainerNode())
     new_end =
         Position(end.ComputeContainerNode(),
                  end.OffsetInContainerNode() - start.OffsetInContainerNode());
   else
     new_end = end;
 
   SplitTextNodeContainingElement(ToText(start.ComputeContainerNode()),
                                  start.OffsetInContainerNode());
-  UpdateStartEnd(Position::BeforeNode(start.ComputeContainerNode()), new_end);
+  UpdateStartEnd(Position::BeforeNode(*start.ComputeContainerNode()), new_end);
 }
 
 void ApplyStyleCommand::SplitTextElementAtEnd(const Position& start,
                                               const Position& end) {
   DCHECK(end.ComputeContainerNode()->IsTextNode()) << end;
 
   bool should_update_start =
       start.ComputeContainerNode() == end.ComputeContainerNode();
   SplitTextNodeContainingElement(ToText(end.ComputeContainerNode()),
                                  end.OffsetInContainerNode());
@@ skipping 223 matching lines @@
 }
 
 Position ApplyStyleCommand::PositionToComputeInlineStyleChange(
     Node* start_node,
     Member<HTMLSpanElement>& dummy_element,
     EditingState* editing_state) {
   // It's okay to obtain the style at the startNode because we've removed all
   // relevant styles from the current run.
   if (!start_node->IsElementNode()) {
     dummy_element = HTMLSpanElement::Create(GetDocument());
-    InsertNodeAt(dummy_element, Position::BeforeNode(start_node),
+    InsertNodeAt(dummy_element, Position::BeforeNode(*start_node),
                  editing_state);
     if (editing_state->IsAborted())
       return Position();
-    return Position::BeforeNode(dummy_element);
+    return Position::BeforeNode(*dummy_element);
   }
 
   return FirstPositionInOrBeforeNode(start_node);
 }
 
 void ApplyStyleCommand::ApplyInlineStyleChange(
     Node* passed_start,
     Node* passed_end,
     StyleChange& style_change,
     EAddStyledElement add_styled_element,
@@ skipping 194 matching lines @@
 
 DEFINE_TRACE(ApplyStyleCommand) {
   visitor->Trace(style_);
   visitor->Trace(start_);
   visitor->Trace(end_);
   visitor->Trace(styled_inline_element_);
   CompositeEditCommand::Trace(visitor);
 }
 
 }  // namespace blink