Make Position::FirstPositionInNode() 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 245 matching lines @@
       visible_end.IsNull() || visible_end.IsOrphan())
     return;
 
   // Save and restore the selection endpoints using their indices in the
   // document, since addBlockStyleIfNeeded may moveParagraphs, which can remove
   // these endpoints. Calculate start and end indices from the start of the tree
   // that they're in.
   Node& scope = NodeTraversal::HighestAncestorOrSelf(
       *visible_start.DeepEquivalent().AnchorNode());
   Range* start_range =
-      Range::Create(GetDocument(), Position::FirstPositionInNode(&scope),
+      Range::Create(GetDocument(), Position::FirstPositionInNode(scope),
                     visible_start.DeepEquivalent().ParentAnchoredEquivalent());
   Range* end_range =
-      Range::Create(GetDocument(), Position::FirstPositionInNode(&scope),
+      Range::Create(GetDocument(), Position::FirstPositionInNode(scope),
                     visible_end.DeepEquivalent().ParentAnchoredEquivalent());
 
   const TextIteratorBehavior behavior =
       TextIteratorBehavior::AllVisiblePositionsRangeLengthBehavior();
 
   int start_index = TextIterator::RangeLength(
       start_range->StartPosition(), start_range->EndPosition(), behavior);
   int end_index = TextIterator::RangeLength(end_range->StartPosition(),
                                             end_range->EndPosition(), behavior);
 
@@ skipping 16 matching lines @@
           paragraph_start.DeepEquivalent();
       if (!remove_only_ && IsEditablePosition(paragraph_start_to_move)) {
         HTMLElement* new_block = MoveParagraphContentsToNewBlockIfNecessary(
             paragraph_start_to_move, editing_state);
         if (editing_state->IsAborted())
           return;
         if (new_block) {
           block = new_block;
           if (paragraph_start.IsOrphan()) {
             GetDocument().UpdateStyleAndLayoutIgnorePendingStylesheets();
-            paragraph_start =
-                CreateVisiblePosition(Position::FirstPositionInNode(new_block));
+            paragraph_start = CreateVisiblePosition(
+                Position::FirstPositionInNode(*new_block));
           }
         }
         DCHECK(!paragraph_start.IsOrphan()) << paragraph_start;
       }
       if (block && block->IsHTMLElement()) {
         RemoveCSSStyle(style, ToHTMLElement(block), editing_state);
         if (editing_state->IsAborted())
           return;
         DCHECK(!paragraph_start.IsOrphan()) << paragraph_start;
         if (!remove_only_) {
@@ skipping 1227 matching lines @@
   if (end.IsOffsetInAnchor() &&
       start.ComputeContainerNode() == end.ComputeContainerNode())
     new_end =
         Position(end.ComputeContainerNode(),
                  end.OffsetInContainerNode() - start.OffsetInContainerNode());
   else
     new_end = end;
 
   Text* text = ToText(start.ComputeContainerNode());
   SplitTextNode(text, start.OffsetInContainerNode());
-  UpdateStartEnd(Position::FirstPositionInNode(text), new_end);
+  UpdateStartEnd(Position::FirstPositionInNode(*text), new_end);
 }
 
 void ApplyStyleCommand::SplitTextAtEnd(const Position& start,
                                        const Position& end) {
   DCHECK(end.ComputeContainerNode()->IsTextNode()) << end;
 
   bool should_update_start =
       start.IsOffsetInAnchor() &&
       start.ComputeContainerNode() == end.ComputeContainerNode();
   Text* text = ToText(end.AnchorNode());
@@ skipping 480 matching lines @@
 
 DEFINE_TRACE(ApplyStyleCommand) {
   visitor->Trace(style_);
   visitor->Trace(start_);
   visitor->Trace(end_);
   visitor->Trace(styled_inline_element_);
   CompositeEditCommand::Trace(visitor);
 }
 
 }  // namespace blink