Fix user pod not always scroll to the center after VK shows up

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 <set>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 238 matching lines @@
   }
   return container_.get();
 }
 
 void KeyboardController::NotifyKeyboardBoundsChanging(
     const gfx::Rect& new_bounds) {
   current_keyboard_bounds_ = new_bounds;
   if (proxy_->HasKeyboardWindow() && proxy_->GetKeyboardWindow()->IsVisible()) {
     FOR_EACH_OBSERVER(KeyboardControllerObserver,
                       observer_list_,
                       OnKeyboardBoundsChanging(new_bounds));

[flackr, 2015/06/24 22:08:42]

Do we ever need to get this Changing notification here or could we just have the
BoundsChanged observer?

[bshe, 2015/06/25 14:01:59]

We probably do. Or we have to make sure the observer is added to observer list
in the right order. See my comment for detail.

     if (keyboard::IsKeyboardOverscrollEnabled()) {
       // Adjust the height of the viewport for visible windows on the primary
       // display.
       // TODO(kevers): Add EnvObserver to properly initialize insets if a
       // window is created while the keyboard is visible.
       scoped_ptr<content::RenderWidgetHostIterator> widgets(
           content::RenderWidgetHost::GetRenderWidgetHosts());
       aura::Window* keyboard_window = proxy_->GetKeyboardWindow();
       aura::Window* root_window = keyboard_window->GetRootWindow();
       while (content::RenderWidgetHost* widget = widgets->GetNextHost()) {
@@ skipping 15 matching lines @@
               view->SetInsets(gfx::Insets(0, 0, overlap, 0));
             else
               view->SetInsets(gfx::Insets());
             AddBoundsChangedObserver(window);
           }
         }
       }
     } else {
       ResetWindowInsets();
     }
+    FOR_EACH_OBSERVER(KeyboardControllerObserver,
+                      observer_list_,
+                      OnKeyboardBoundsChanged(new_bounds));
   } else {
     current_keyboard_bounds_ = gfx::Rect();
   }
 }
 
 void KeyboardController::HideKeyboard(HideReason reason) {
   keyboard_visible_ = false;
   ToggleTouchEventLogging(true);
 
   keyboard::LogKeyboardControlEvent(
@@ skipping 290 matching lines @@
   proxy_->HideKeyboardContainer(container_.get());
 }
 
 void KeyboardController::AddBoundsChangedObserver(aura::Window* window) {
   aura::Window* target_window = window ? window->GetToplevelWindow() : nullptr;
   if (target_window)
     window_bounds_observer_->AddObservedWindow(target_window);
 }
 
 }  // namespace keyboard