Virtual Keyboard: Update insets on change to window bounds.

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 "base/bind.h"
 #include "base/command_line.h"
 #include "content/public/browser/render_widget_host.h"
 #include "content/public/browser/render_widget_host_iterator.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "ui/aura/window.h"
 #include "ui/aura/window_delegate.h"
+#include "ui/aura/window_observer.h"
 #include "ui/base/cursor/cursor.h"
 #include "ui/base/hit_test.h"
 #include "ui/base/ime/input_method.h"
 #include "ui/base/ime/text_input_client.h"
 #include "ui/compositor/layer_animation_observer.h"
 #include "ui/compositor/scoped_layer_animation_settings.h"
 #include "ui/gfx/path.h"
 #include "ui/gfx/rect.h"
 #include "ui/gfx/skia_util.h"
 #include "ui/keyboard/keyboard_controller_observer.h"
@@ skipping 158 matching lines @@
     return;
   animator_->RemoveObserver(this);
   callback_.Run();
 }
 
 void CallbackAnimationObserver::OnLayerAnimationAborted(
     ui::LayerAnimationSequence* seq) {
   animator_->RemoveObserver(this);
 }
 
+class WindowBoundsChangeObserver : public aura::WindowObserver {
+ public:
+  virtual void OnWindowBoundsChanged(aura::Window* window,
+                                     const gfx::Rect& old_bounds,
+                                     const gfx::Rect& new_bounds) OVERRIDE;
+};
+
+void WindowBoundsChangeObserver::OnWindowBoundsChanged(aura::Window* window,
+    const gfx::Rect& old_bounds, const gfx::Rect& new_bounds) {
+  KeyboardController* controller =  KeyboardController::GetInstance();
+  if (controller)
+    controller->UpdateWindowInsets(window);
+}
+
 // static
 KeyboardController* KeyboardController::instance_ = NULL;
 
 KeyboardController::KeyboardController(KeyboardControllerProxy* proxy)
     : proxy_(proxy),
       input_method_(NULL),
       keyboard_visible_(false),
       lock_keyboard_(false),
       type_(ui::TEXT_INPUT_TYPE_NONE),
       weak_factory_(this) {
   CHECK(proxy);
   input_method_ = proxy_->GetInputMethod();
   input_method_->AddObserver(this);
+  window_bounds_observer_.reset(new WindowBoundsChangeObserver());
 }
 
 KeyboardController::~KeyboardController() {
   if (container_)
     container_->RemoveObserver(this);
   if (input_method_)
     input_method_->RemoveObserver(this);
   ResetWindowInsets();
 }
 
@@ skipping 49 matching lines @@
           if (window != keyboard_window &&
               window->GetRootWindow() == root_window) {
             gfx::Rect window_bounds = window->GetBoundsInScreen();
             gfx::Rect intersect = gfx::IntersectRects(window_bounds,
                                                       new_bounds);
             int overlap = intersect.height();
             if (overlap > 0 && overlap < window_bounds.height())
               view->SetInsets(gfx::Insets(0, 0, overlap, 0));
             else
               view->SetInsets(gfx::Insets(0, 0, 0, 0));
-            // TODO(kevers): Add window observer to native window to update
-            // insets on a window move or resize.
+            AddBoundsChangedObserver(window);
           }
         }
       }
     } else {
       ResetWindowInsets();
     }
   }
 }
 
 void KeyboardController::HideKeyboard(HideReason reason) {
@@ skipping 91 matching lines @@
 void KeyboardController::OnInputMethodDestroyed(
     const ui::InputMethod* input_method) {
   DCHECK_EQ(input_method_, input_method);
   input_method_ = NULL;
 }
 
 void KeyboardController::OnShowImeIfNeeded() {
   ShowKeyboardInternal();
 }
 
+void KeyboardController::UpdateWindowInsets(aura::Window* window) {
+  aura::Window *keyboard_window = proxy_->GetKeyboardWindow();
+  if (window == keyboard_window)
+    return;
+
+  bool enableInsets = (keyboard_window->GetRootWindow() ==
+      window->GetRootWindow()) && keyboard::IsKeyboardOverscrollEnabled() &&
+      proxy_->HasKeyboardWindow() && proxy_->GetKeyboardWindow()->IsVisible();

[bshe, 2014/07/04 13:03:57]

nit: Do you need HasKeyboardWindow? It should always be true after you call
GetKeyboardWindow above.

[kevers, 2014/07/18 14:06:23]

Done.

+
+  scoped_ptr<content::RenderWidgetHostIterator> widgets(
+      content::RenderWidgetHost::GetRenderWidgetHosts());
+  while (content::RenderWidgetHost* widget = widgets->GetNextHost()) {
+    content::RenderWidgetHostView* view = widget->GetView();
+    if (view && window->Contains(view->GetNativeView())) {
+      gfx::Rect window_bounds = view->GetNativeView()->GetBoundsInScreen();
+      gfx::Rect intersect = gfx::IntersectRects(window_bounds,
+          proxy_->GetKeyboardWindow()->bounds());
+      int overlap = enableInsets ? intersect.height() : 0;
+      if (overlap > 0 && overlap < window_bounds.height())
+        view->SetInsets(gfx::Insets(0, 0, overlap, 0));
+      else
+        view->SetInsets(gfx::Insets(0, 0, 0, 0));

[bshe, 2014/07/04 13:03:57]

nit: probably gfx::Insets()?

[kevers, 2014/07/18 14:06:23]

Done.

+      return;
+    }
+  }
+}
+
 void KeyboardController::ShowKeyboardInternal() {
   if (!container_.get())
     return;
 
   if (container_->children().empty()) {
     keyboard::MarkKeyboardLoadStarted();
     aura::Window* keyboard = proxy_->GetKeyboardWindow();
     keyboard->Show();
     container_->AddChild(keyboard);
     keyboard->set_owned_by_parent(false);
@@ skipping 56 matching lines @@
     container_->layer()->SetOpacity(1.0);
   }
 }
 
 void KeyboardController::ResetWindowInsets() {
   const gfx::Insets insets;
   scoped_ptr<content::RenderWidgetHostIterator> widgets(
       content::RenderWidgetHost::GetRenderWidgetHosts());
   while (content::RenderWidgetHost* widget = widgets->GetNextHost()) {
     content::RenderWidgetHostView* view = widget->GetView();
-    if (view)
+    if (view) {
       view->SetInsets(insets);
+      aura::Window *window = view->GetNativeView();
+      RemoveBoundsChangedObserver(window);
+    }
   }
 }
 
 bool KeyboardController::WillHideKeyboard() const {
   return weak_factory_.HasWeakPtrs();
 }
 
 void KeyboardController::ShowAnimationFinished() {
   // Notify observers after animation finished to prevent reveal desktop
   // background during animation.
   NotifyKeyboardBoundsChanging(proxy_->GetKeyboardWindow()->bounds());
   proxy_->EnsureCaretInWorkArea();
 }
 
 void KeyboardController::HideAnimationFinished() {
   proxy_->HideKeyboardContainer(container_.get());
 }
 
+void KeyboardController::AddBoundsChangedObserver(aura::Window* window) {
+  while(window->parent() && window->parent()->id() < 0) {

[bshe, 2014/07/04 13:03:57]

nit: do you mind to add a comment on why do we check "id < 0"? Or maybe add an
anonymous function since this while loop doesn't depend on KeyboardController
and is used in more than one place.

[kevers, 2014/07/18 14:06:23]

Done.

+    window = window->parent();
+  }
+  if (!window->HasObserver(window_bounds_observer_.get()))
+    window->AddObserver(window_bounds_observer_.get());
+}
+
+void KeyboardController::RemoveBoundsChangedObserver(aura::Window* window) {
+  while(window->parent() && window->parent()->id() < 0) {
+    window = window->parent();
+  }
+  if (window && window->HasObserver(window_bounds_observer_.get()))

[bshe, 2014/07/04 13:03:57]

nit: you probably dont need to check if window is null in this if check or you
should probably check it before the while loop.

[kevers, 2014/07/18 14:06:23]

Done.

+    window->RemoveObserver(window_bounds_observer_.get());
+}
+
 }  // namespace keyboard