Audit the use of UpdateStyleAndLayoutIgnorePendingStylesheets in FrameSelection::revealSelection

third_party/WebKit/Source/core/editing/FrameSelection.cpp

Index: third_party/WebKit/Source/core/editing/FrameSelection.cpp
diff --git a/third_party/WebKit/Source/core/editing/FrameSelection.cpp b/third_party/WebKit/Source/core/editing/FrameSelection.cpp
index 7564787ddb425c49a371a7f18a959716513d1ee2..b7a58c980fa5e560d238fcaa815eaffb6e707d5e 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelection.cpp
@@ -335,6 +335,10 @@ void FrameSelection::setSelectionAlgorithm(const VisibleSelectionTemplate<Strate
         else
             alignment = (align == CursorAlignOnScroll::Always) ? ScrollAlignment::alignTopAlways : ScrollAlignment::alignToEdgeIfNeeded;
 
+        // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
+        // needs to be audited.  See http://crbug.com/590369 for more details.
+        document().updateStyleAndLayoutIgnorePendingStylesheets();
+
         revealSelection(alignment, RevealExtent);
     }
 
@@ -1152,10 +1156,7 @@ HTMLFormElement* FrameSelection::currentForm() const
 void FrameSelection::revealSelection(const ScrollAlignment& alignment, RevealExtentOption revealExtentOption)
 {
     DCHECK(isAvailable());
-
-    // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-    // needs to be audited.  See http://crbug.com/590369 for more details.
-    document().updateStyleAndLayoutIgnorePendingStylesheets();
+    DCHECK(!document().needsLayoutTreeUpdate());
 
     LayoutRect rect;