Remove unnecessary layout updates from InsertInto/DeleteFromTextNodeCommand

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

 /*
  * Copyright (C) 2005, 2006, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2009, 2010, 2011 Google 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
@@ skipping 932 matching lines @@
   // Merging forward could result in deleting the destination anchor node.
   // To avoid this, we add a placeholder node before the start of the paragraph.
   if (endOfParagraph(startOfParagraphToMove).deepEquivalent() ==
       destination.deepEquivalent()) {
     HTMLBRElement* placeholder = HTMLBRElement::create(document());
     insertNodeBefore(placeholder,
                      startOfParagraphToMove.deepEquivalent().anchorNode(),
                      editingState);
     if (editingState->isAborted())
       return;
+
+    // TODO(editing-dev): Use of updateStyleAndLayoutIgnorePendingStylesheets()
+    // needs to be audited.  See http://crbug.com/590369 for more details.
     document().updateStyleAndLayoutIgnorePendingStylesheets();
+
     destination = VisiblePosition::beforeNode(placeholder);
     startOfParagraphToMove =
         createVisiblePosition(startOfParagraphToMove.toPositionWithAffinity());
   }
 
   moveParagraph(startOfParagraphToMove, endOfParagraph(startOfParagraphToMove),
                 destination, editingState);
   if (editingState->isAborted())
     return;
 
@@ skipping 1017 matching lines @@
       mostForwardCaretPosition(endingSelection().end()).anchorNode();
   Text* textNode = toText(fragment.firstChild());
   // Our fragment creation code handles tabs, spaces, and newlines, so we don't
   // have to worry about those here.
 
   Position start = endingSelection().start();
   Position end = replaceSelectedTextInNode(textNode->data());
   if (end.isNull())
     return false;
 
+  document().updateStyleAndLayoutIgnorePendingStylesheets();
+
   if (nodeAfterInsertionPos && nodeAfterInsertionPos->parentNode() &&
       isHTMLBRElement(*nodeAfterInsertionPos) &&
       shouldRemoveEndBR(toHTMLBRElement(nodeAfterInsertionPos),
                         VisiblePosition::beforeNode(nodeAfterInsertionPos))) {
     removeNodeAndPruneAncestors(nodeAfterInsertionPos, editingState);
     if (editingState->isAborted())
       return false;
   }
 
   m_startOfInsertedRange = start;
@@ skipping 19 matching lines @@
   visitor->trace(m_startOfInsertedContent);
   visitor->trace(m_endOfInsertedContent);
   visitor->trace(m_insertionStyle);
   visitor->trace(m_documentFragment);
   visitor->trace(m_startOfInsertedRange);
   visitor->trace(m_endOfInsertedRange);
   CompositeEditCommand::trace(visitor);
 }
 
 }  // namespace blink