Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(66)

Unified Diff: chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc

Issue 2293183002: ash: Remove ash::Shelf in favor of ash::WmShelf (Closed)
Patch Set: rebase again Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc
diff --git a/chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc b/chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc
index 9f2229c6eaf58c2ed712a66a144b1c784cb96878..cad2fe6307841bc94534d7674f60f5d4bf46c59b 100644
--- a/chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc
+++ b/chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.cc
@@ -5,7 +5,6 @@
#include "chrome/browser/ui/ash/multi_user/user_switch_animator_chromeos.h"
#include "ash/aura/wm_window_aura.h"
-#include "ash/common/shelf/shelf.h"
#include "ash/common/shelf/shelf_layout_manager.h"
#include "ash/common/shelf/shelf_widget.h"
#include "ash/common/shelf/wm_shelf.h"
@@ -234,7 +233,7 @@ void UserSwitchAnimatorChromeOS::TransitionUserShelf(
new_account_id_.GetUserEmail());
// Hide the black rectangle on top of each shelf again.
for (ash::WmWindow* window : ash::WmShell::Get()->GetAllRootWindows()) {
- ash::ShelfWidget* shelf = ash::Shelf::ForWindow(window)->shelf_widget();
+ ash::ShelfWidget* shelf = ash::WmShelf::ForWindow(window)->shelf_widget();
shelf->HideShelfBehindBlackBar(false, duration_override);
}
// We kicked off the shelf animation above and the override can be
@@ -266,7 +265,8 @@ void UserSwitchAnimatorChromeOS::TransitionUserShelf(
// CPU usage and therefore effect jank, we should avoid hiding the shelf if
// the start and end location are the same and cover the shelf instead with
// a black rectangle on top.
- ash::Shelf* shelf = ash::Shelf::ForWindow(ash::WmWindowAura::Get(window));
+ ash::WmShelf* shelf =
+ ash::WmShelf::ForWindow(ash::WmWindowAura::Get(window));
if (GetScreenCover(window) != NO_USER_COVERS_SCREEN &&
(!chrome_launcher_controller ||
!chrome_launcher_controller->ShelfBoundsChangesProbablyWithUser(
@@ -276,8 +276,7 @@ void UserSwitchAnimatorChromeOS::TransitionUserShelf(
// This shelf change is only part of the animation and will be updated by
// ChromeLauncherController::ActiveUserChanged() to the new users value.
// Note that the user preference will not be changed.
- shelf->wm_shelf()->SetAutoHideBehavior(
- ash::SHELF_AUTO_HIDE_ALWAYS_HIDDEN);
+ shelf->SetAutoHideBehavior(ash::SHELF_AUTO_HIDE_ALWAYS_HIDDEN);
}
}
}
« no previous file with comments | « chrome/browser/ui/ash/multi_user/multi_user_window_manager_chromeos.cc ('k') | chrome/browser/ui/ash/shelf_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698