Introduce computePositionForNodeRemoval() as replacement of updatePositionForNodeRemoval()

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

 /*
  * Copyright (C) 2004, 2008, 2009, 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 173 matching lines @@
   const Position& newExtent =
       computePositionForChildrenRemoval(oldExtent, container);
   if (newBase == oldBase && newExtent == oldExtent)
     return;
   m_selection = SelectionInDOMTree::Builder()
                     .setBaseAndExtent(newBase, newExtent)
                     .build();
   markCacheDirty();
 }
 
-static Position computePositionForNodeRemoval(const Position& position,
-                                              Node& nodeToBeRemoved) {
-  Position result = position;
-  // TODO(yosin): We should rename |updatePositionForNodeRemoval()|
-  // to |computePositionForNodeRemoval()| to avoid using output parameter.
-  updatePositionForNodeRemoval(result, nodeToBeRemoved);
-  return result;
-}
-
 void SelectionEditor::nodeWillBeRemoved(Node& nodeToBeRemoved) {
   if (m_selection.isNone())
     return;
   const Position oldBase = m_selection.m_base;
   const Position oldExtent = m_selection.m_extent;
   const Position& newBase =
       computePositionForNodeRemoval(oldBase, nodeToBeRemoved);
   const Position& newExtent =
       computePositionForNodeRemoval(oldExtent, nodeToBeRemoved);
   if (newBase == oldBase && newExtent == oldExtent)
@@ skipping 221 matching lines @@
   visitor->trace(m_frame);
   visitor->trace(m_selection);
   visitor->trace(m_cachedVisibleSelectionInDOMTree);
   visitor->trace(m_cachedVisibleSelectionInFlatTree);
   visitor->trace(m_logicalRange);
   visitor->trace(m_cachedRange);
   SynchronousMutationObserver::trace(visitor);
 }
 
 }  // namespace blink