[InputEvent] Update EndingSelection when appending UndoStep

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

Index: third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
diff --git a/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp b/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
index 2e7e96569faf363e2f3101d17d433167ef652201..d97958470a1a57c88bdbf3b58a39d4018ee91e11 100644
--- a/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
+++ b/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp
@@ -216,6 +216,8 @@ void CompositeEditCommand::ApplyCommandToComposite(
 void CompositeEditCommand::AppendCommandToUndoStep(
     CompositeEditCommand* command) {
   EnsureUndoStep()->Append(command->EnsureUndoStep());
+  EnsureUndoStep()->SetEndingSelection(

[yosin_UTC9, 2017/05/08 04:01:05]

Could you move |SetEndingSelection()| call to |Editor::AppliedEditing()| after
|AppendCommandToUndoStep()|?
To make |AppendCommandToUndoStep()| does just appending?
I'm not sure whether |AppendCommandToUndoStep()| should update ending selection
or not.
I would like to postpone the decision until we have another call site.

[chongz, 2017/05/08 23:37:53]

Moved to |Editor::AppliedEditing()| but I have to call |SetEndingSelection()|
before |AppendCommandToUndoStep()|.
e.g. |command->undo_step_| will be cleared during |AppendCommandToUndoStep()|.

+      command->EnsureUndoStep()->EndingSelection());
   command->undo_step_ = nullptr;
   command->SetParent(this);
   commands_.push_back(command);