Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in FrameSelection::selectedText

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 81b629de313f190fcb135211c1cd5dd35fcacc89..ac9d459816810b6aaf652f7d52ec8d9bc9ef4590 100644
--- a/third_party/WebKit/Source/core/editing/FrameSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/FrameSelection.cpp
@@ -1071,13 +1071,6 @@ String FrameSelection::selectedHTMLForClipboard() const
 
 String FrameSelection::selectedText(TextIteratorBehavior behavior) const
 {
-    if (!isAvailable())

[yosin_UTC9, 2016/09/07 09:58:25]

Could you have |DCHECK(isAvailable())|, just in case?

[Xiaocheng, 2016/09/07 10:01:56]

This |isAvailable()| check was added by myself a few days ago for hoisting the
layout update call from |plainText()|.

If anything goes wrong, it will be caught in |plainText()|.

-        return emptyString();
-
-    // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-    // needs to be audited.  See http://crbug.com/590369 for more details.
-    m_document->updateStyleAndLayoutIgnorePendingStylesheets();
-
     return extractSelectedText(*this, behavior);
 }