Chromium Code Reviews| Index: content/browser/renderer_host/render_widget_host_view_android.cc |
| diff --git a/content/browser/renderer_host/render_widget_host_view_android.cc b/content/browser/renderer_host/render_widget_host_view_android.cc |
| index 53a05f24fa4956bde4f2b93542a860a08a10e2d1..54f80b07ab3a9f6f4a2ba02da8096a3861f470c4 100644 |
| --- a/content/browser/renderer_host/render_widget_host_view_android.cc |
| +++ b/content/browser/renderer_host/render_widget_host_view_android.cc |
| @@ -404,11 +404,11 @@ int RenderWidgetHostViewAndroid::GetNativeImeAdapter() { |
| void RenderWidgetHostViewAndroid::OnTextInputStateChanged( |
| const ViewHostMsg_TextInputState_Params& params) { |
| #if defined(OS_ANDROID) |
| - if (params.require_ack) { |
| - // Regardless of how we exit from this method, we must acknowledge that we |
| - // processed the input state change. |
| - base::ScopedClosureRunner ack_caller(base::Bind(&SendImeEventAck, host_)); |
| - } |
| + // If an acknowledgement is required for this event, regardless of how we exit |
| + // from this method, we must acknowledge that we processed the input state |
| + // change. |
| + base::ScopedClosureRunner ack_caller(base::Bind(&SendImeEventAck, host_)); |
| + if (!params.require_ack) ack_caller.Release(); |
|
cjhopman
2013/08/09 00:20:35
Nit: add a break after condition
|
| #endif |
| if (!IsShowing()) |
| return; |