Make WebViewImpl::textInputInfo update layout before working on ranges.

third_party/WebKit/Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 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 2420 matching lines @@
     info.inputMode = inputModeOfFocusedElement();
 
     info.type = textInputType();
     info.flags = textInputFlags();
     if (info.type == WebTextInputTypeNone)
         return info;
 
     if (!focused->editor().canEdit())
         return info;
 
+    // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
+    // see http://crbug.com/590369 for more details.
+    focused->document()->updateStyleAndLayoutIgnorePendingStylesheets();
+
+    DocumentLifecycle::DisallowTransitionScope(focused->document()->lifecycle());

[Xiaocheng, 2016/08/30 15:18:24]

Pass-by: I think we need to have a named scope here, like what
crrev.com/2297503002 does. Otherwise the scope just gets destructed immediately.

+
     // Emits an object replacement character for each replaced element so that
     // it is exposed to IME and thus could be deleted by IME on android.
     info.value = plainText(EphemeralRange::rangeOfContents(*element), TextIteratorEmitsObjectReplacementCharacter);
 
     if (info.value.isEmpty())
         return info;
 
     EphemeralRange firstRange = firstEphemeralRangeOf(selection.selection());
     if (firstRange.isNotNull()) {
         PlainTextRange plainTextRange(PlainTextRange::create(*element, firstRange));
@@ skipping 2149 matching lines @@
         return nullptr;
     return focusedFrame;
 }
 
 LocalFrame* WebViewImpl::focusedLocalFrameAvailableForIme() const
 {
     return m_imeAcceptEvents ? focusedLocalFrameInWidget() : nullptr;
 }
 
 } // namespace blink