Blimp: Added spinner to text input dialog

blimp/engine/feature/engine_render_widget_feature.cc

Index: blimp/engine/feature/engine_render_widget_feature.cc
diff --git a/blimp/engine/feature/engine_render_widget_feature.cc b/blimp/engine/feature/engine_render_widget_feature.cc
index fccb811d2a62988b8a5bed826622bdf92535fb34..0728b132493c33827992e22d839ef39423969a3e 100644
--- a/blimp/engine/feature/engine_render_widget_feature.cc
+++ b/blimp/engine/feature/engine_render_widget_feature.cc
@@ -231,6 +231,15 @@ void EngineRenderWidgetFeature::ProcessMessage(
       if (render_widget_host && render_widget_host->GetView()) {
         SetTextFromIME(render_widget_host->GetView()->GetTextInputClient(),
                        message->ime().ime_text());
+
+        // TODO(shaktisahu): Remove this fake HIDE_IME request once the blimp
+        // IME design is completed.
+        base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(

[David Trainor- moved to gerrit, 2016/10/19 16:40:31]

Actually I think it's cleaner if we send this delayed from the client.  Can we
delay how we hit the callback by 0.5 seconds or something?  That already has a
callback_.Run(), now it'll be PostDelayedTask(callback_, blah blah,
base::TimeDelta::FromMilliseconds(500))?

[shaktisahu, 2016/10/27 02:42:56]

Actually, today, engine is not sending a HIDE_IME request which will cause the
spinner to spin indefinitely.

[David Trainor- moved to gerrit, 2016/10/28 05:08:08]

Yeah I was assuming we'd just send the hide right away, but it's a good point
that we can just add the delay here since we already have to call the hide
method.

Can you file a bug and tag it here?

[shaktisahu, 2016/11/01 21:35:33]

Done.

+            FROM_HERE,
+            base::Bind(&EngineRenderWidgetFeature::SendHideImeRequest,
+                       base::Unretained(this), target_tab_id,
+                       render_widget_host),
+            base::TimeDelta::FromMilliseconds(1500));
       }
       break;
     default: