DRAFT: Add setKeyboardVisibility API.

ui/keyboard/keyboard_controller.cc

 // Copyright (c) 2013 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 "ui/keyboard/keyboard_controller.h"
 
 #include "ui/aura/layout_manager.h"
 #include "ui/aura/window.h"
 #include "ui/aura/window_delegate.h"
 #include "ui/base/hit_test.h"
@@ skipping 124 matching lines @@
   if (!container_) {
     container_ = new aura::Window(new KeyboardWindowDelegate());
     container_->SetName("KeyboardContainer");
     container_->Init(ui::LAYER_NOT_DRAWN);
     container_->AddObserver(this);
     container_->SetLayoutManager(new KeyboardLayoutManager(container_));
   }
   return container_;
 }
 
-void KeyboardController::AddObserver(KeyboardControllerObserver* observer) {
-  observer_list_.AddObserver(observer);
-}
-
-void KeyboardController::RemoveObserver(KeyboardControllerObserver* observer) {
-  observer_list_.RemoveObserver(observer);
-}
-
-void KeyboardController::OnWindowParentChanged(aura::Window* window,
-                                               aura::Window* parent) {
-  OnTextInputStateChanged(proxy_->GetInputMethod()->GetTextInputClient());
-}
-
-void KeyboardController::OnWindowDestroying(aura::Window* window) {
-  DCHECK_EQ(container_, window);
-  container_ = NULL;
-}
-
-void KeyboardController::OnTextInputStateChanged(
-    const ui::TextInputClient* client) {
+void KeyboardController::SetKeyboardVisibility(bool should_show) {
   if (!container_)
     return;
 
   bool was_showing = container_->IsVisible();
-  bool should_show = was_showing;
-  if (!client || client->GetTextInputType() == ui::TEXT_INPUT_TYPE_NONE) {
-    should_show = false;
-  } else {
-    if (container_->children().empty()) {
+  if (was_showing != should_show) {
+    if (should_show && container_->children().empty()) {
       aura::Window* keyboard = proxy_->GetKeyboardWindow();
       keyboard->Show();
       container_->AddChild(keyboard);
       container_->layout_manager()->OnWindowResized();
     }
-    container_->parent()->StackChildAtTop(container_);
-    should_show = true;
-  }
 
-  if (was_showing != should_show) {
     gfx::Rect new_bounds(
         should_show ? container_->children()[0]->bounds() : gfx::Rect());
 
     FOR_EACH_OBSERVER(
         KeyboardControllerObserver,
         observer_list_,
         OnKeyboardBoundsChanging(new_bounds));
 
-    if (should_show)
+    if (should_show) {
+      container_->parent()->StackChildAtTop(container_);
       container_->Show();
-    else
+    } else {
       container_->Hide();

[bryeung, 2013/04/30 20:21:03]

directly hiding the keyboard from JS seems to cause jank: the animation on the
button looks to still be in progress when the keyboard is shown again.  Any idea
what's going on with that?

[sadrul, 2013/05/14 06:23:19]

Animation in the page itself should not be causing the window-hide to jank.
Something is probably doing bad things, but I can't think of what it might be.

+    }
 
     proxy_->OnKeyboardBoundsChanged(new_bounds);
   }
+}
+
+void KeyboardController::AddObserver(KeyboardControllerObserver* observer) {
+  observer_list_.AddObserver(observer);
+}
+
+void KeyboardController::RemoveObserver(KeyboardControllerObserver* observer) {
+  observer_list_.RemoveObserver(observer);
+}
+
+void KeyboardController::OnWindowParentChanged(aura::Window* window,
+                                               aura::Window* parent) {
+  OnTextInputStateChanged(proxy_->GetInputMethod()->GetTextInputClient());
+}
+
+void KeyboardController::OnWindowDestroying(aura::Window* window) {
+  DCHECK_EQ(container_, window);
+  container_ = NULL;
+}
+
+void KeyboardController::OnTextInputStateChanged(
+    const ui::TextInputClient* client) {
+  SetKeyboardVisibility(
+      (client && client->GetTextInputType() != ui::TEXT_INPUT_TYPE_NONE));
 
   // TODO(bryeung): whenever the TextInputClient changes we need to notify the
   // keyboard (with the TextInputType) so that it can reset it's state (e.g.
   // abandon compositions in progress)
 }
 
 void KeyboardController::OnInputMethodDestroyed(
     const ui::InputMethod* input_method) {
   DCHECK_EQ(input_method_, input_method);
   input_method_ = NULL;
 }
 
 }  // namespace keyboard