Chromium Code Reviews| 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 7de5e8b39c254e8afce7e2232790e6e07d8469e4..c53b1db60b2a4fbcc39c7088c0ce49e10af8519e 100644 |
| --- a/blimp/engine/feature/engine_render_widget_feature.cc |
| +++ b/blimp/engine/feature/engine_render_widget_feature.cc |
| @@ -14,6 +14,7 @@ |
| #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 "content/public/common/form_field_data.h" |
| #include "net/base/net_errors.h" |
| #include "third_party/WebKit/public/web/WebInputEvent.h" |
| #include "ui/events/event.h" |
| @@ -135,9 +136,7 @@ void EngineRenderWidgetFeature::SendCompositorMessage( |
| void EngineRenderWidgetFeature::SendShowImeRequest( |
| const int tab_id, |
| content::RenderWidgetHost* render_widget_host, |
| - const ui::TextInputClient* client) { |
| - DCHECK(client); |
| - |
| + const content::FormFieldData& field) { |
|
David Trainor- moved to gerrit
2016/11/17 23:05:41
This will probably encapsulate popups eventually t
|
| ImeMessage* ime_message; |
| std::unique_ptr<BlimpMessage> blimp_message = |
| CreateBlimpMessage(&ime_message, tab_id); |
| @@ -147,13 +146,9 @@ void EngineRenderWidgetFeature::SendShowImeRequest( |
| ime_message->set_render_widget_id(render_widget_id); |
| ime_message->set_type(ImeMessage::SHOW_IME); |
| ime_message->set_text_input_type( |
| - InputMessageConverter::TextInputTypeToProto(client->GetTextInputType())); |
| - |
| - gfx::Range text_range; |
| - base::string16 existing_text; |
| - client->GetTextRange(&text_range); |
| - client->GetTextFromRange(text_range, &existing_text); |
| - ime_message->set_ime_text(base::UTF16ToUTF8(existing_text)); |
| + InputMessageConverter::TextInputTypeToProto(field.text_input_type)); |
| + ime_message->set_ime_text(field.text); |
| + // TODO(shaktisahu): Add remaining fields to proto. |
| ime_message_sender_->ProcessMessage(std::move(blimp_message), |
| net::CompletionCallback()); |