Blimp: Added spinner to text input dialog

blimp/engine/feature/engine_render_widget_feature.cc

 // Copyright 2015 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 "blimp/engine/feature/engine_render_widget_feature.h"
 
 #include "base/numerics/safe_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "blimp/common/create_blimp_message.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/common/proto/compositor.pb.h"
 #include "blimp/common/proto/input.pb.h"
 #include "blimp/common/proto/render_widget.pb.h"
 #include "blimp/net/input_message_converter.h"
 #include "content/public/browser/render_widget_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "net/base/net_errors.h"
 #include "third_party/WebKit/public/web/WebInputEvent.h"
 #include "ui/events/event.h"
 #include "ui/events/keycodes/dom/dom_code.h"
 
 namespace blimp {
 namespace engine {
 
 EngineRenderWidgetFeature::EngineRenderWidgetFeature(SettingsManager* settings)
-    : settings_manager_(settings) {
+    : settings_manager_(settings), weak_factory_(this) {
   DCHECK(settings_manager_);
   settings_manager_->AddObserver(this);
 }
 
 EngineRenderWidgetFeature::~EngineRenderWidgetFeature() {
   DCHECK(settings_manager_);
   settings_manager_->RemoveObserver(this);
 }
 
 void EngineRenderWidgetFeature::set_render_widget_message_sender(
@@ skipping 189 matching lines @@
         delegate->OnCompositorMessageReceived(render_widget_host, payload);
       }
       break;
     case BlimpMessage::kIme:
       DCHECK(message->ime().type() == ImeMessage::SET_TEXT);
       render_widget_host =
           GetRenderWidgetHost(target_tab_id, message->ime().render_widget_id());
       if (render_widget_host && render_widget_host->GetView()) {
         SetTextFromIME(render_widget_host, message->ime().ime_text(),
                        message->ime().auto_submit());
+
+        // TODO(shaktisahu): Remove this fake HIDE_IME request once the blimp
+        // IME design is completed (crbug/661328).
+        base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
+            FROM_HERE,
+            base::Bind(&EngineRenderWidgetFeature::SendHideImeRequest,
+                       weak_factory_.GetWeakPtr(), target_tab_id,
+                       render_widget_host),
+            base::TimeDelta::FromMilliseconds(1500));

[Wez, 2016/11/07 22:51:52]

What's the rationale for the delay here?

       }
       break;
     default:
       NOTREACHED();
   }
 
   callback.Run(net::OK);
 }
 
 void EngineRenderWidgetFeature::OnWebPreferencesChanged() {
@@ skipping 140 matching lines @@
   IdToRenderWidgetMap::const_iterator widget_id_it =
       id_to_render_widget->find(render_widget_id);
   if (widget_id_it == id_to_render_widget->end())
     return nullptr;
 
   return widget_id_it->second;
 }
 
 }  // namespace engine
 }  // namespace blimp