Commands preserve handles

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

 /*
  * Copyright (C) 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * 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
@@ skipping 1481 matching lines @@
 void Editor::ChangeSelectionAfterCommand(
     const SelectionInDOMTree& new_selection,
     FrameSelection::SetSelectionOptions options) {
   if (new_selection.IsNone())
     return;
 
   // See <rdar://problem/5729315> Some shouldChangeSelectedDOMRange contain
   // Ranges for selections that are no longer valid
   bool selection_did_not_change_dom_position =
       new_selection == GetFrame().Selection().GetSelectionInDOMTree();
-  GetFrame().Selection().SetSelection(new_selection, options);
+  GetFrame().Selection().SetSelection(
+      SelectionInDOMTree::Builder(new_selection)
+          .SetIsHandleVisible(GetFrame().Selection().IsHandleVisible())
+          .Build(),
+      options);
 
   // Some editing operations change the selection visually without affecting its
   // position within the DOM. For example when you press return in the following
   // (the caret is marked by ^):
   // <div contentEditable="true"><div>^Hello</div></div>
   // WebCore inserts <div><br></div> *before* the current block, which correctly
   // moves the paragraph down but which doesn't change the caret's DOM position
   // (["hello", 0]). In these situations the above FrameSelection::setSelection
   // call does not call EditorClient::respondToChangedSelection(), which, on the
   // Mac, sends selection change notifications and starts a new kill ring
@@ skipping 334 matching lines @@
 
 DEFINE_TRACE(Editor) {
   visitor->Trace(frame_);
   visitor->Trace(last_edit_command_);
   visitor->Trace(undo_stack_);
   visitor->Trace(mark_);
   visitor->Trace(typing_style_);
 }
 
 }  // namespace blink