FrameSelection::textWasReplaced and setSelectedRange shouldn't trigger synchronous layout

Source/core/editing/FrameSelection.cpp

Index: Source/core/editing/FrameSelection.cpp
diff --git a/Source/core/editing/FrameSelection.cpp b/Source/core/editing/FrameSelection.cpp
index b17d02cffe1fd1c102dda897746bccb9de644ac7..ac3ccc5a65ff6b2c5d4d958d4a8c06317312d0d5 100644
--- a/Source/core/editing/FrameSelection.cpp
+++ b/Source/core/editing/FrameSelection.cpp
@@ -471,7 +471,6 @@ void FrameSelection::updateSelectionIfNeeded(const Position& base, const Positio
         return;
     VisibleSelection newSelection;
     newSelection.setWithoutValidation(base, extent);
-    m_frame->document()->updateLayout();
     setSelection(newSelection, DoNotSetFocus);
 }
 
@@ -1409,17 +1408,13 @@ bool FrameSelection::setSelectedRange(Range* range, EAffinity affinity, SetSelec
         return false;
     ASSERT(range->startContainer()->document() == range->endContainer()->document());
 
-    m_frame->document()->updateLayoutIgnorePendingStylesheets();
-
     // Non-collapsed ranges are not allowed to start at the end of a line that is wrapped,
     // they start at the beginning of the next line instead
     m_logicalRange = nullptr;
     stopObservingVisibleSelectionChangeIfNecessary();
 
-    // FIXME: Can we provide extentAffinity?
-    VisiblePosition visibleStart(range->startPosition(), range->collapsed() ? affinity : DOWNSTREAM);
-    VisiblePosition visibleEnd(range->endPosition(), SEL_DEFAULT_AFFINITY);
-    setSelection(VisibleSelection(visibleStart, visibleEnd), options);
+    VisibleSelection newSelection(range, affinity);
+    setSelection(newSelection, options);
 
     m_logicalRange = range->cloneRange();
     startObservingVisibleSelectionChange();