Fix OSK flickering issue

content/renderer/render_widget.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/render_widget.h"
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 1872 matching lines @@
 #if defined(OS_ANDROID)
   UpdateTextInputState(NO_SHOW_IME, FROM_IME);
 #endif
 }
 
 void RenderWidget::UpdateTextInputState(ShowIme show_ime,
                                         ChangeSource change_source) {
   TRACE_EVENT0("renderer", "RenderWidget::UpdateTextInputState");
   if (handling_ime_event_)
     return;
-  ui::TextInputType new_type = GetTextInputType();
+
+#if defined(OS_ANDROID)

[aelias_OOO_until_Jul13, 2015/10/15 19:58:40]

I don't think we should make this kind of change behind a platform branch, it
makes the flow too hard to reason about.  Let's try to ship the task posting
approach on all platforms.

[Changwan Ryu, 2015/10/16 02:10:16]

Done.

+  // We could be called in the middle of focus change, or in some other
+  // intermediary state.
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&RenderWidget::UpdateTextInputStateInternal, this,
+                            show_ime, change_source));
+#else
+  UpdateTextInputStateInternal(show_ime, change_source);
+#endif  // defined(OS_ANDROID)
+}
+
+void RenderWidget::UpdateTextInputStateInternal(ShowIme show_ime,
+                                                ChangeSource change_source) {
+  blink::WebTextInputInfo new_info;
+  if (webwidget_)
+    new_info = webwidget_->textInputInfo();
+
+  ui::TextInputType new_type = WebKitToUiTextInputType(new_info.type);
   if (IsDateTimeInput(new_type))
     return;  // Not considered as a text input field in WebKit/Chromium.
 
-  blink::WebTextInputInfo new_info;
-  if (webwidget_)
-    new_info = webwidget_->textInputInfo();
   const ui::TextInputMode new_mode = ConvertInputMode(new_info.inputMode);
 
   bool new_can_compose_inline = CanComposeInline();
 
   // Only sends text input params if they are changed or if the ime should be
   // shown.
   if (show_ime == SHOW_IME_IF_NEEDED ||
       (text_input_type_ != new_type ||
        text_input_mode_ != new_mode ||
        text_input_info_ != new_info ||
@@ skipping 475 matching lines @@
 void RenderWidget::RegisterVideoHoleFrame(RenderFrameImpl* frame) {
   video_hole_frames_.AddObserver(frame);
 }
 
 void RenderWidget::UnregisterVideoHoleFrame(RenderFrameImpl* frame) {
   video_hole_frames_.RemoveObserver(frame);
 }
 #endif  // defined(VIDEO_HOLE)
 
 }  // namespace content