Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in toNormalizedEphemeralRange

third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp

Index: third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
diff --git a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
index 6234fcd4d0646a20a47a6f8b52f45a47e70962c5..bc2f7358acf3330626b29253f26ab437e8d9c4bd 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp
@@ -991,14 +991,14 @@ bool SpellChecker::selectionStartHasSpellingMarkerFor(int from,
 
 void SpellChecker::removeMarkers(const VisibleSelection& selection,
                                  DocumentMarker::MarkerTypes markerTypes) {
-  const EphemeralRange range = selection.toNormalizedEphemeralRange();
-  if (range.isNull())
-    return;
-
   // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
   // needs to be audited.  See http://crbug.com/590369 for more details.
   frame().document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
+  const EphemeralRange range = selection.toNormalizedEphemeralRange();

[yosin_UTC9, 2016/10/14 08:13:00]

s/const EphemeralRange/const EphemeralRange&/

[Xiaocheng, 2016/10/14 08:35:14]

Done.

+  if (range.isNull())
+    return;
+
   frame().document()->markers().removeMarkers(range, markerTypes);
 }