Move CompositeEditCommand::deleteSelection() to TypingCommand::deleteSelectionIfRange()

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

Index: third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h
diff --git a/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h b/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h
index bee9c60634d12b2fae70f6c49e9e8f3570d13801..beb582b4ef935859ea9d8979340500a716440601 100644
--- a/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h
+++ b/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.h
@@ -105,12 +105,14 @@ class CORE_EXPORT CompositeEditCommand : public EditCommand {
                        bool mergeBlocksAfterDelete = true,
                        bool expandForSpecialElements = true,
                        bool sanitizeMarkup = true);
-  void deleteSelection(const VisibleSelection&,
-                       EditingState*,
-                       bool smartDelete = false,
-                       bool mergeBlocksAfterDelete = true,
-                       bool expandForSpecialElements = true,
-                       bool sanitizeMarkup = true);
+  // TODO(yosin): We should move |deleteSelectionIfRange()| to |TypingCommand|
+  // class since it is used in |TypingCommand| only.

[yoichio, 2017/03/07 05:16:31]

Could you move now instead of renaming?

+  void deleteSelectionIfRange(const VisibleSelection&,
+                              EditingState*,
+                              bool smartDelete = false,
+                              bool mergeBlocksAfterDelete = true,
+                              bool expandForSpecialElements = true,
+                              bool sanitizeMarkup = true);
   virtual void deleteTextFromNode(Text*, unsigned offset, unsigned count);
   bool isRemovableBlock(const Node*);
   void insertNodeAfter(Node*, Node* refChild, EditingState*);