Extracting placeholder information from Webkit to Blimp

blimp/engine/session/tab.cc

 // Copyright 2016 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/session/tab.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "base/trace_event/trace_event.h"
 #include "blimp/common/create_blimp_message.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/common/proto/render_widget.pb.h"
 #include "blimp/common/proto/tab_control.pb.h"
 #include "blimp/engine/common/blimp_user_agent.h"
 #include "blimp/engine/feature/engine_render_widget_feature.h"
 #include "blimp/net/blimp_message_processor.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/render_widget_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/web_contents.h"
+#include "content/public/common/form_field_data.h"
 #include "content/public/common/renderer_preferences.h"
 #include "ui/aura/window.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace blimp {
 namespace engine {
 
+namespace {
+
+// Tracks the number of text input requests.
+static int g_text_input_request_id = 0;
+
+}  // namespace
+
 Tab::Tab(std::unique_ptr<content::WebContents> web_contents,
          const int tab_id,
          EngineRenderWidgetFeature* render_widget_feature,
          BlimpMessageProcessor* navigation_message_sender)
     : web_contents_(std::move(web_contents)),
       tab_id_(tab_id),
       render_widget_feature_(render_widget_feature),
       navigation_message_sender_(navigation_message_sender),
-      page_load_tracker_(web_contents_.get(), this) {
+      page_load_tracker_(web_contents_.get(), this),
+      weak_factory_(this) {
   DCHECK(render_widget_feature_);
   DCHECK(navigation_message_sender_);
 
   // A Tab is created upon client's request, thus an updated
   // user agent info (containing client OS info) is available, and we will use
   // that to override user agent string from BlimpContentRendererClient.
   web_contents_->SetUserAgentOverride(GetBlimpEngineUserAgent());
 
   render_widget_feature_->SetDelegate(tab_id_, this);
 
@@ skipping 113 matching lines @@
   navigation_message_sender_->ProcessMessage(std::move(message),
                                              net::CompletionCallback());
 }
 
 void Tab::OnWebGestureEvent(content::RenderWidgetHost* render_widget_host,
                             std::unique_ptr<blink::WebGestureEvent> event) {
   TRACE_EVENT1("blimp", "Tab::OnWebGestureEvent", "type", event->type);
   render_widget_host->ForwardGestureEvent(*event);
 }
 
+void Tab::ShowTextInputUI() {
+  g_text_input_request_id++;
+  content::FormFieldDataCallback callback =
+      base::Bind(&Tab::ProcessTextInputInfo, weak_factory_.GetWeakPtr(),
+                 g_text_input_request_id);
+
+  content::RenderFrameHost* focused_frame = web_contents()->GetFocusedFrame();
+  if (focused_frame) {
+    focused_frame->RequestFocusedFormFieldData(callback);
+  }
+}
+
+void Tab::HideTextInputUI() {
+  g_text_input_request_id++;
+  render_widget_feature_->SendHideImeRequest(
+      tab_id(),
+      web_contents()->GetRenderWidgetHostView()->GetRenderWidgetHost());
+}
+
+void Tab::ProcessTextInputInfo(int request_id,
+                               const content::FormFieldData& field) {
+  if (field.text_input_type == ui::TEXT_INPUT_TYPE_NONE)

[Charlie Reis, 2016/11/16 20:32:49]

Do you still need this?  Probably better to remove unless there's a case for it.

[shaktisahu, 2016/11/17 01:16:15]

Yes, we still return TEXT_INPUT_TYPE_NONE when the renderer cannot obtain the
information about the focused text field correctly.

+    return;
+
+  // Discard the results for old requests.
+  if (request_id < g_text_input_request_id) {

[Charlie Reis, 2016/11/16 20:32:49]

Does this need to be global?  Seems like a member would make more sense.

[shaktisahu, 2016/11/17 01:16:15]

I am not sure. I felt like this is cleaner. Making it a class member will
properly reset it though. dtrainor@ - What do you think?

[David Trainor- moved to gerrit, 2016/11/17 23:05:41]

A member sounds good IMO.  current_form_request_id_?

+    return;
+  }
+
+  // TODO(shaktisahu): Remove adding RenderWidgetHost info to the proto.
+  render_widget_feature_->SendShowImeRequest(
+      tab_id(),
+      web_contents()->GetRenderWidgetHostView()->GetRenderWidgetHost(), field);
+}
+
 void Tab::OnCompositorMessageReceived(
     content::RenderWidgetHost* render_widget_host,
     const std::vector<uint8_t>& message) {
   TRACE_EVENT0("blimp", "Tab::OnCompositorMessageReceived");
 
   render_widget_host->HandleCompositorProto(message);
 }
 
 }  // namespace engine
 }  // namespace blimp