Utilize EditCommand::setEndingSelection() taking SelectionInDOMTree

third_party/WebKit/Source/core/editing/commands/EditCommand.h

 /*
  * Copyright (C) 2005, 2006, 2008 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 51 matching lines @@
 
   DECLARE_VIRTUAL_TRACE();
 
  protected:
   explicit EditCommand(Document&);
 
   Document& document() const { return *m_document.get(); }
   CompositeEditCommand* parent() const { return m_parent; }
   void setStartingSelection(const VisibleSelection&);
   void setEndingSelection(const SelectionInDOMTree&);
-  // TODO(yosin): We should get rid of overloads of |setEndingSelection()|
-  // except for taking |SelectionInDOMTree|.
-  void setEndingSelection(const VisibleSelection&);
+  // TODO(yosin): |setEndingVisibleSelection()| will take |SelectionInUndoStep|
+  // You should not use this function other than copying existing selection.
+  void setEndingVisibleSelection(const VisibleSelection&);
 
   // TODO(yosin) |isRenderedCharacter()| should be removed, and we should use
   // |VisiblePosition::characterAfter()| and
   // |VisiblePosition::characterBefore()|.
   static bool isRenderedCharacter(const Position&);
 
  private:
   Member<Document> m_document;
   VisibleSelection m_startingSelection;
   VisibleSelection m_endingSelection;
@@ skipping 19 matching lines @@
 
 DEFINE_TYPE_CASTS(SimpleEditCommand,
                   EditCommand,
                   command,
                   command->isSimpleEditCommand(),
                   command.isSimpleEditCommand());
 
 }  // namespace blink
 
 #endif  // EditCommand_h