Fix the full screen switching browser window dipping below launcher bar issue.

ash/wm/workspace_controller.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 "ash/wm/workspace_controller.h"
 
 #include "ash/shell.h"
 #include "ash/wm/window_util.h"
 #include "ash/wm/workspace/workspace_event_filter.h"
 #include "ash/wm/workspace/workspace_layout_manager.h"
@@ skipping 57 matching lines @@
           views::MenuItemView::TOPRIGHT, views::MenuRunner::HAS_MNEMONICS) ==
       views::MenuRunner::MENU_DELETED)
     return;
 #endif  // !defined(OS_MACOSX)
 }
 
 void WorkspaceController::OnRootWindowResized(const gfx::Size& new_size) {
   workspace_manager_->SetWorkspaceSize(new_size);
 }
 
+void WorkspaceController::OnScreenWorkAreaInsetsChanged() {
+  workspace_manager_->SetWorkspaceBounds();

[sky, 2012/03/01 01:43:19]

Shouldn't this be the same as OnRootWindowResized?

[jennyz, 2012/03/01 18:27:45]

OnScreenWorkAreaInsetsChanged() does not pass the new_size of the workspace in.
OnRootWindowResized needs to pass in the new_size of the work space, and set its
work_space_ to new_size, which we don't want it to be done for work area insets
change case, the work space does not change.

[sky, 2012/03/01 18:43:45]

I see what you're saying. In that case name the WorkspaceManager method the same
as this and have both it and SetWorkspaceSize call into a new method called
SetWorkspaceBounds.

[jennyz, 2012/03/01 19:13:12]

Done.

+}
+
 void WorkspaceController::OnWindowPropertyChanged(aura::Window* window,
                                                   const void* key,
                                                   intptr_t old) {
   if (key == aura::client::kRootWindowActiveWindowKey)
     workspace_manager_->SetActiveWorkspaceByWindow(wm::GetActiveWindow());
 }
 
 bool WorkspaceController::IsCommandIdChecked(int command_id) const {
   switch (static_cast<MenuItem>(command_id)) {
     case MENU_SNAP_TO_GRID:
@@ skipping 46 matching lines @@
 }
 
 bool WorkspaceController::GetAcceleratorForCommandId(
       int command_id,
       ui::Accelerator* accelerator) {
   return false;
 }
 
 }  // namespace internal
 }  // namespace ash