Commands preserve handles

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 176 matching lines @@
   const Position old_base = selection_.base_;
   const Position old_extent = selection_.extent_;
   const Position& new_base =
       ComputePositionForChildrenRemoval(old_base, container);
   const Position& new_extent =
       ComputePositionForChildrenRemoval(old_extent, container);
   if (new_base == old_base && new_extent == old_extent)
     return;
   selection_ = SelectionInDOMTree::Builder()
                    .SetBaseAndExtent(new_base, new_extent)
+                   .SetIsHandleVisible(selection_.IsHandleVisible())

[yosin_UTC9, 2017/05/31 03:39:26]

Good catch! (^_^)b

                    .Build();
   MarkCacheDirty();
 }
 
 void SelectionEditor::NodeWillBeRemoved(Node& node_to_be_removed) {
   if (selection_.IsNone())
     return;
   const Position old_base = selection_.base_;
   const Position old_extent = selection_.extent_;
   const Position& new_base =
       ComputePositionForNodeRemoval(old_base, node_to_be_removed);
   const Position& new_extent =
       ComputePositionForNodeRemoval(old_extent, node_to_be_removed);
   if (new_base == old_base && new_extent == old_extent)
     return;
   selection_ = SelectionInDOMTree::Builder()
                    .SetBaseAndExtent(new_base, new_extent)
+                   .SetIsHandleVisible(selection_.IsHandleVisible())
                    .Build();
   MarkCacheDirty();
 }
 
 static Position UpdatePositionAfterAdoptingTextReplacement(
     const Position& position,
     CharacterData* node,
     unsigned offset,
     unsigned old_length,
     unsigned new_length) {
@@ skipping 218 matching lines @@
 DEFINE_TRACE(SelectionEditor) {
   visitor->Trace(frame_);
   visitor->Trace(selection_);
   visitor->Trace(cached_visible_selection_in_dom_tree_);
   visitor->Trace(cached_visible_selection_in_flat_tree_);
   visitor->Trace(cached_range_);
   SynchronousMutationObserver::Trace(visitor);
 }
 
 }  // namespace blink