Remove layout updates from Editor::{applied,unapplied,reapplied}Editing

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 778 matching lines @@
     startRoot->dispatchEvent(
         Event::create(EventTypeNames::webkitEditableContentChanged));
   if (endRoot && endRoot != startRoot)
     endRoot->dispatchEvent(
         Event::create(EventTypeNames::webkitEditableContentChanged));
 }
 
 void Editor::appliedEditing(CompositeEditCommand* cmd) {
   DCHECK(!cmd->isCommandGroupWrapper());
   EventQueueScope scope;
-  frame().document()->updateStyleAndLayout();
 
   // Request spell checking before any further DOM change.
   spellChecker().markMisspellingsAfterApplyingCommand(*cmd);
 
   EditCommandComposition* composition = cmd->composition();
   DCHECK(composition);
   dispatchEditableContentChangedEvents(
       composition->startingRootEditableElement(),
       composition->endingRootEditableElement());
   // TODO(chongz): Filter empty InputType after spec is finalized.
@@ skipping 36 matching lines @@
       !passedSelection.extent().isConnected())
     return VisibleSelection();
   DCHECK(!passedSelection.base().document()->needsLayoutTreeUpdate());
   VisibleSelection correctedSelection = passedSelection;
   correctedSelection.updateIfNeeded();
   return correctedSelection;
 }
 
 void Editor::unappliedEditing(EditCommandComposition* cmd) {
   EventQueueScope scope;
-  frame().document()->updateStyleAndLayout();
 
   dispatchEditableContentChangedEvents(cmd->startingRootEditableElement(),
                                        cmd->endingRootEditableElement());
   dispatchInputEventEditableContentChanged(
       cmd->startingRootEditableElement(), cmd->endingRootEditableElement(),
       InputEvent::InputType::Undo, nullAtom,
       InputEvent::EventIsComposing::NotComposing);
 
   // TODO(editing-dev): The use of updateStyleAndLayoutIgnorePendingStylesheets
   // needs to be audited.  See http://crbug.com/590369 for more details.
   // In the long term, we should stop editing commands from storing
   // VisibleSelections as starting and ending selections.
   frame().document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
   VisibleSelection newSelection =
       correctedVisibleSelection(cmd->startingSelection());
   if (newSelection.start().document() == frame().document() &&
       newSelection.end().document() == frame().document())
     changeSelectionAfterCommand(
         newSelection,
         FrameSelection::CloseTyping | FrameSelection::ClearTypingStyle);
 
   m_lastEditCommand = nullptr;
   m_undoStack->registerRedoStep(cmd);
   respondToChangedContents(newSelection);
 }
 
 void Editor::reappliedEditing(EditCommandComposition* cmd) {
   EventQueueScope scope;
-  frame().document()->updateStyleAndLayout();
 
   dispatchEditableContentChangedEvents(cmd->startingRootEditableElement(),
                                        cmd->endingRootEditableElement());
   dispatchInputEventEditableContentChanged(
       cmd->startingRootEditableElement(), cmd->endingRootEditableElement(),
       InputEvent::InputType::Redo, nullAtom,
       InputEvent::EventIsComposing::NotComposing);
 
   // TODO(yosin): Since |dispatchEditableContentChangedEvents()| and
   // |dispatchInputEventEditableContentChanged()|, we would like to know
@@ skipping 747 matching lines @@
 }
 
 DEFINE_TRACE(Editor) {
   visitor->trace(m_frame);
   visitor->trace(m_lastEditCommand);
   visitor->trace(m_undoStack);
   visitor->trace(m_mark);
 }
 
 }  // namespace blink