Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in toNormalizedEphemeralRange

third_party/WebKit/Source/web/ContextMenuClientImpl.cpp

 /*
  * Copyright (C) 2009, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 286 matching lines @@
 
   if (selectedFrame != m_webView->page()->mainFrame()) {
     data.frameURL = urlFromFrame(selectedFrame);
     HistoryItem* historyItem = selectedFrame->loader().currentItem();
     if (historyItem)
       data.frameHistoryItem = WebHistoryItem(historyItem);
   }
 
   if (r.isSelected()) {
     if (!isHTMLInputElement(*r.innerNode()) ||
-        toHTMLInputElement(r.innerNode())->type() != InputTypeNames::password)
+        toHTMLInputElement(r.innerNode())->type() != InputTypeNames::password) {
+      // TODO(xiaochengh): Use of updateStyleAndLayoutIgnorePendingStylesheets
+      // needs to be audited.  See http://crbug.com/590369 for more details.
+      // Plain text extraction requires clean layout.
+      selectedFrame->document()->updateStyleAndLayoutIgnorePendingStylesheets();
+
       data.selectedText = selectedFrame->selectedText().stripWhiteSpace();
+    }
   }
 
   if (r.isContentEditable()) {
     data.isEditable = true;
 
     // Spellchecker adds spelling markers to misspelled words and attaches
     // suggestions to these markers in the background. Therefore, when a
     // user right-clicks a mouse on a word, Chrome just needs to find a
     // spelling marker on the word instead of spellchecking it.
     String description;
@@ skipping 129 matching lines @@
   subMenuItems.swap(outputItems);
 }
 
 void ContextMenuClientImpl::populateCustomMenuItems(
     const ContextMenu* defaultMenu,
     WebContextMenuData* data) {
   populateSubMenuItems(defaultMenu->items(), data->customItems);
 }
 
 }  // namespace blink