Blimp: IME should submit form with text

blimp/client/core/contents/ime_feature.cc

Index: blimp/client/core/contents/ime_feature.cc
diff --git a/blimp/client/core/contents/ime_feature.cc b/blimp/client/core/contents/ime_feature.cc
index 4a4bcdbfe331165cdd44125d5d47c0e95aab0ac9..97b633c894973f11587916c76e3a8445953289f5 100644
--- a/blimp/client/core/contents/ime_feature.cc
+++ b/blimp/client/core/contents/ime_feature.cc
@@ -14,13 +14,20 @@
 namespace blimp {
 namespace client {
 
+ImeFeature::WebInputRequest::WebInputRequest()
+    : input_type(ui::TEXT_INPUT_TYPE_NONE) {}
+
+ImeFeature::WebInputRequest::~WebInputRequest() = default;
+
+ImeFeature::WebInputResponse::WebInputResponse() : submit(false) {}
+
 ImeFeature::ImeFeature() {}
 
 ImeFeature::~ImeFeature() {}
 
 void ImeFeature::OnImeTextEntered(int tab_id,
                                   int render_widget_id,
-                                  const std::string& text) {
+                                  const WebInputResponse& response) {
   DCHECK_LE(0, tab_id);
   DCHECK_LT(0, render_widget_id);
 
@@ -29,7 +36,8 @@ void ImeFeature::OnImeTextEntered(int tab_id,
       CreateBlimpMessage(&ime_message, tab_id);
   ime_message->set_render_widget_id(render_widget_id);
   ime_message->set_type(ImeMessage::SET_TEXT);
-  ime_message->set_ime_text(text);
+  ime_message->set_ime_text(response.text);
+  ime_message->set_auto_submit(response.submit);
 
   outgoing_message_processor_->ProcessMessage(std::move(blimp_message),
                                               net::CompletionCallback());
@@ -52,13 +60,15 @@ void ImeFeature::ProcessMessage(std::unique_ptr<BlimpMessage> message,
         return;
       }
       {
-        ShowImeCallback show_ime_callback = base::Bind(
+        WebInputRequest request;
+        request.input_type = InputMessageConverter::TextInputTypeFromProto(
+            ime_message.text_input_type());
+        request.text = ime_message.ime_text();
+        request.show_ime_callback = base::Bind(
             &ImeFeature::OnImeTextEntered, base::Unretained(this),

[David Trainor- moved to gerrit, 2016/10/18 16:53:45]

Hmm base::Unretained() seems incorrect here.  We should probably be using a
WeakPtr.  If this class dies while we have a request out we don't want to call
back into it.

[shaktisahu, 2016/10/18 19:15:59]

Done.

             message->target_tab_id(), ime_message.render_widget_id());
-        delegate_->OnShowImeRequested(
-            InputMessageConverter::TextInputTypeFromProto(
-                ime_message.text_input_type()),
-            ime_message.ime_text(), show_ime_callback);
+
+        delegate_->OnShowImeRequested(request);
       }
       break;
     case ImeMessage::HIDE_IME: