Improve how DocumentMarkerController updates markers in response to text edits

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

 /*
  * Copyright (C) 2005 Apple Computer, 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 742 matching lines @@
   // TODO(yosin) |isRenderedCharacter()| should be removed, and we should use
   // |VisiblePosition::characterAfter()| and
   // |VisiblePosition::characterBefore()|
   if (leading_whitespace_.IsNotNull() &&
       !IsRenderedCharacter(leading_whitespace_) &&
       leading_whitespace_.AnchorNode()->IsTextNode()) {
     Text* text_node = ToText(leading_whitespace_.AnchorNode());
     DCHECK(!text_node->GetLayoutObject() ||
            text_node->GetLayoutObject()->Style()->CollapseWhiteSpace())
         << text_node;
-    ReplaceTextInNodePreservingMarkers(
-        text_node, leading_whitespace_.ComputeOffsetInContainerNode(), 1,
-        NonBreakingSpaceString());
+    ReplaceTextInNode(text_node,
+                      leading_whitespace_.ComputeOffsetInContainerNode(), 1,
+                      NonBreakingSpaceString());
   }
   if (trailing_whitespace_.IsNotNull() &&
       !IsRenderedCharacter(trailing_whitespace_) &&
       trailing_whitespace_.AnchorNode()->IsTextNode()) {
     Text* text_node = ToText(trailing_whitespace_.AnchorNode());
     DCHECK(!text_node->GetLayoutObject() ||
            text_node->GetLayoutObject()->Style()->CollapseWhiteSpace())
         << text_node;
-    ReplaceTextInNodePreservingMarkers(
-        text_node, trailing_whitespace_.ComputeOffsetInContainerNode(), 1,
-        NonBreakingSpaceString());
+    ReplaceTextInNode(text_node,
+                      trailing_whitespace_.ComputeOffsetInContainerNode(), 1,
+                      NonBreakingSpaceString());
   }
 }
 
 // If a selection starts in one block and ends in another, we have to merge to
 // bring content before the start together with content after the end.
 void DeleteSelectionCommand::MergeParagraphs(EditingState* editing_state) {
   if (!merge_blocks_after_delete_) {
     if (prune_start_block_if_necessary_) {
       // We aren't going to merge into the start block, so remove it if it's
       // empty.
@@ skipping 457 matching lines @@
   visitor->Trace(delete_into_blockquote_style_);
   visitor->Trace(start_root_);
   visitor->Trace(end_root_);
   visitor->Trace(start_table_row_);
   visitor->Trace(end_table_row_);
   visitor->Trace(temporary_placeholder_);
   CompositeEditCommand::Trace(visitor);
 }
 
 }  // namespace blink