Implement virtual keyboard hiding.

content/browser/renderer_host/render_widget_host_view_aura.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/render_widget_host_view_aura.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
@@ skipping 2571 matching lines @@
   }
 
   switch (event->type()) {
     case ui::ET_MOUSE_PRESSED:
       window_->SetCapture();
       // Confirm existing composition text on mouse click events, to make sure
       // the input caret won't be moved with an ongoing composition text.
       FinishImeCompositionSession();
       break;
     case ui::ET_MOUSE_RELEASED:
+      // The input method needs to know if an already-focused text field was
+      // clicked again (refocused).
+      GetInputMethod()->OnPossibleRefocus();
+

[SteveT, 2013/07/31 03:19:18]

Handling "refocus" here and below.

[Seigo Nonaka, 2013/08/07 08:41:18]

Hmm, this might work for many cases but I think this approach is not a straight
forward way.
I think we should introduce OnActiveInputTargetChanged() callback instead of
OnTextInputTypeChanged() into Blink (and also omnibox), then propagate to the
IME.
But this approach requires big work for many places, so I think TODO is fine for
at this moment.

Also please note that input focus can be changed by JavaScript or TAB key.

On 2013/07/31 03:19:18, SteveT wrote:

[SteveT, 2013/08/07 12:26:39]

Just Omnibox, or every other native textbox, like the find bar?

But yes, it seems bad to have this change in such a fundamental part of the UI
code, but it also seems like other IME event stuff is handled here.
I'll introduce the TODO after I see what kevers@ does about the API namespace...

And then after this lands I'll start an email thread with you and you can guide
me through the changes needed in Blink (where I'm not familiar).

       window_->ReleaseCapture();
       break;
     default:
       break;
   }
 
   // Needed to propagate mouse event to native_tab_contents_view_aura.
   // TODO(pkotwicz): Find a better way of doing this.
   // In fullscreen mode which is typically used by flash, don't forward
   // the mouse events to the parent. The renderer and the plugin process
@@ skipping 31 matching lines @@
   }
 
   event->SetHandled();
 }
 
 void RenderWidgetHostViewAura::OnTouchEvent(ui::TouchEvent* event) {
   TRACE_EVENT0("input", "RenderWidgetHostViewAura::OnTouchEvent");
   if (touch_editing_client_ && touch_editing_client_->HandleInputEvent(event))
     return;
 
+  if (event->type() == ui::ET_TOUCH_RELEASED)
+    GetInputMethod()->OnPossibleRefocus();
+
   // Update the touch event first.
   WebKit::WebTouchPoint* point = UpdateWebTouchEventFromUIEvent(*event,
                                                                 &touch_event_);
 
   // Forward the touch event only if a touch point was updated, and there's a
   // touch-event handler in the page, and no other touch-event is in the queue.
   // It is important to always consume the event if there is a touch-event
   // handler in the page, or some touch-event is already in the queue, even if
   // no point has been updated, to make sure that this event does not get
   // processed by the gesture recognizer before the events in the queue.
@@ skipping 515 matching lines @@
     RenderWidgetHost* widget) {
   return new RenderWidgetHostViewAura(widget);
 }
 
 // static
 void RenderWidgetHostViewPort::GetDefaultScreenInfo(WebScreenInfo* results) {
   GetScreenInfoForWindow(results, NULL);
 }
 
 }  // namespace content