Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in PlainTextRange.

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

 /*
  * Copyright (C) 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 381 matching lines @@
 
     // In case of exceeding the left boundary.
     int selectionOffsetsStart = static_cast<int>(getSelectionOffsets().start());
     int start = std::max(selectionOffsetsStart + selectionStart, 0);
     int end = std::max(selectionOffsetsStart + selectionEnd, start);
 
     Element* rootEditableElement = frame().selection().rootEditableElement();
     if (!rootEditableElement)
         return;
 
+    // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
+    // see http://crbug.com/590369 for more details.
+    rootEditableElement->document().updateStyleAndLayoutIgnorePendingStylesheets();
+
     // In case of exceeding the right boundary.
     // If both |value1| and |value2| exceed right boundary,
     // PlainTextRange(value1, value2)::createRange() will return a default
     // value, which is [0,0]. In order to get the correct Position in that case,
     // we should make sure |value1| is within range at least.
     const EphemeralRange& startRange = PlainTextRange(0, start).createRange(*rootEditableElement);
     const EphemeralRange& endRange = PlainTextRange(0, end).createRange(*rootEditableElement);
 
     // TODO(yabinh): There should be a better way to create |startPosition| and
     // |endPosition|. But for now, since we can't get |anchorNode| and |offset|,
@@ skipping 17 matching lines @@
         frame().document()->markers().addCompositionMarker(ephemeralLineRange.startPosition(), ephemeralLineRange.endPosition(), underline.color, underline.thick, underline.backgroundColor);
     }
 }
 
 void InputMethodController::setCompositionFromExistingText(const Vector<CompositionUnderline>& underlines, unsigned compositionStart, unsigned compositionEnd)
 {
     Element* editable = frame().selection().rootEditableElement();
     if (!editable)
         return;
 
+    // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
+    // see http://crbug.com/590369 for more details.
+    editable->document().updateStyleAndLayoutIgnorePendingStylesheets();
+
     const EphemeralRange range = PlainTextRange(compositionStart, compositionEnd).createRange(*editable);
     if (range.isNull())
         return;
 
     const Position start = range.startPosition();
     if (rootEditableElementOf(start) != editable)
         return;
 
     const Position end = range.endPosition();
     if (rootEditableElementOf(end) != editable)
@@ skipping 45 matching lines @@
 }
 
 bool InputMethodController::setSelectionOffsets(const PlainTextRange& selectionOffsets)
 {
     if (selectionOffsets.isNull())
         return false;
     Element* rootEditableElement = frame().selection().rootEditableElement();
     if (!rootEditableElement)
         return false;
 
+    // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
+    // see http://crbug.com/590369 for more details.
+    rootEditableElement->document().updateStyleAndLayoutIgnorePendingStylesheets();
+
     const EphemeralRange range = selectionOffsets.createRange(*rootEditableElement);
     if (range.isNull())
         return false;
 
     return frame().selection().setSelectedRange(range, VP_DEFAULT_AFFINITY, SelectionDirectionalMode::NonDirectional, FrameSelection::CloseTyping);
 }
 
 bool InputMethodController::setEditableSelectionOffsets(const PlainTextRange& selectionOffsets)
 {
     if (!editor().canEdit())
@@ skipping 34 matching lines @@
     TypingCommand::deleteSelection(*frame().document());
 }
 
 DEFINE_TRACE(InputMethodController)
 {
     visitor->trace(m_frame);
     visitor->trace(m_compositionRange);
 }
 
 } // namespace blink