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

Unified Diff: ash/root_window_controller.h

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
« no previous file with comments | « ash/mus/workspace/workspace_layout_manager_unittest.cc ('k') | ash/root_window_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/root_window_controller.h
diff --git a/ash/root_window_controller.h b/ash/root_window_controller.h
index 2f26f0dff61c741c0953f5b3c5bc2a5c6f0dec73..7481d308425ecb4c3049b49bd344e50e0b607e36 100644
--- a/ash/root_window_controller.h
+++ b/ash/root_window_controller.h
@@ -55,7 +55,6 @@ class AnimatingWallpaperWidgetController;
class DockedWindowLayoutManager;
enum class LoginStatus;
class PanelLayoutManager;
-class Shelf;
class ShelfLayoutManager;
class ShelfWidget;
class StackingController;
@@ -186,10 +185,6 @@ class ASH_EXPORT RootWindowController : public ShellObserver {
// Creates the shelf for this root window and notifies observers.
void CreateShelf();
- // Returns the shelf controller for this root window.
- // TODO(jamescook): Remove this and use WmRootWindowController::GetShelf().
- Shelf* GetShelf() const;
-
// Called when the login status changes after login (such as lock/unlock).
// TODO(oshima): Investigate if we can merge this and |OnLoginStateChanged|.
void UpdateAfterLoginStatusChange(LoginStatus status);
@@ -280,10 +275,8 @@ class ASH_EXPORT RootWindowController : public ShellObserver {
// to it during construction of the shelf widget and status tray.
std::unique_ptr<WmShelfAura> wm_shelf_aura_;
- // Legacy shelf controller. Only present after shelf is created (post-login).
- std::unique_ptr<Shelf> shelf_;
-
// The shelf widget for this root window.
+ // TODO(jamescook): Move ownership to WmShelf.
std::unique_ptr<ShelfWidget> shelf_widget_;
// An invisible/empty window used as a event target for
« no previous file with comments | « ash/mus/workspace/workspace_layout_manager_unittest.cc ('k') | ash/root_window_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698