New accessibility virtual keyboard behavior in non-sticky mode.

ui/base/ime/input_method_base.cc

Index: ui/base/ime/input_method_base.cc
diff --git a/ui/base/ime/input_method_base.cc b/ui/base/ime/input_method_base.cc
index 9a37f8dfde652adc7b930b5d823687e6c3fcb79d..531a7b3dcb3571613b26e435ef9174b4393df26d 100644
--- a/ui/base/ime/input_method_base.cc
+++ b/ui/base/ime/input_method_base.cc
@@ -59,6 +59,12 @@ TextInputClient* InputMethodBase::GetTextInputClient() const {
   return text_input_client_;
 }
 
+void InputMethodBase::set_available_bounds(gfx::Rect new_bounds) {
+  available_bounds_ = new_bounds;
+  if (text_input_client_)
+    text_input_client_->EnsureCaretInRect(available_bounds_);
+}
+
 void InputMethodBase::OnTextInputTypeChanged(const TextInputClient* client) {
   if (!IsTextInputClientFocused(client))
     return;
@@ -141,6 +147,18 @@ void InputMethodBase::SetFocusedTextInputClientInternal(
   text_input_client_ = client;  // nullptr allowed.
   OnDidChangeFocusedClient(old, client);
   NotifyTextInputStateChanged(text_input_client_);
+
+  if (available_bounds_.IsEmpty())
+    return;
+
+  // Restore old focused window if moved.
+  if (old) {

[sky, 2016/08/19 00:16:40]

In general chrome style is no {} around single line ifs. For example, see line
144/145 above.

[hariank, 2016/08/19 00:45:38]

Done.

+    old->EnsureCaretInRect(available_bounds_);

[sky, 2016/08/19 00:16:40]

Why do you need to notify the previous TextInputClient here?

[hariank, 2016/08/19 00:45:38]

The previous TextInputClient's window should be restored if it was moved. When
the keyboard is closed, the active (most recently moved) window is restored, but
not all the previously moved windows. So, whenever focus is changed the previous
window is restored.

[sky, 2016/08/19 03:47:24]

That makes sense. I would never expect a function named EnsureCaretInRect to
handle restore. You should add an explicit function for this to TextInputClient.
Maybe something like OnNoLongerActiveTextInputClient. That names is wordy, but
you get the idea.

[hariank, 2016/08/19 17:54:39]

Ok, added an OnClientFocusLost function.

+  }
+  // Move new focused window if necessary.
+  if (text_input_client_) {
+    text_input_client_->EnsureCaretInRect(available_bounds_);
+  }
 }
 
 std::vector<gfx::Rect> InputMethodBase::GetCompositionBounds(