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

Unified Diff: ash/mus/window_manager.cc

Issue 2625733003: Re-reland: chromeos: Fix shelf appearing at login screen under mash (Closed)
Patch Set: remove include Created 3 years, 11 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: ash/mus/window_manager.cc
diff --git a/ash/mus/window_manager.cc b/ash/mus/window_manager.cc
index 5946488013fd18246e613b9e236117fec04072cf..f3ebdc9a05cf635fbf1d779b9ede3030af118c81 100644
--- a/ash/mus/window_manager.cc
+++ b/ash/mus/window_manager.cc
@@ -8,6 +8,7 @@
#include <utility>
+#include "ash/common/session/session_controller.h"
#include "ash/common/wm/container_finder.h"
#include "ash/common/wm/window_state.h"
#include "ash/display/screen_position_controller.h"
@@ -134,9 +135,6 @@ void WindowManager::Init(
this, pointer_watcher_event_router_.get()));
shell_->Initialize(blocking_pool);
lookup_.reset(new WmLookupMus);
-
- // TODO: this should be called when logged in. See http://crbug.com/654606.
- shell_->CreateShelf();
}
aura::client::ActivationClient* WindowManager::activation_client() {
@@ -222,8 +220,9 @@ RootWindowController* WindowManager::CreateRootWindowController(
root_window_controller_ptr.get();
root_window_controllers_.insert(std::move(root_window_controller_ptr));
- // TODO: this should be called when logged in. See http://crbug.com/654606.
- root_window_controller->wm_root_window_controller()->CreateShelf();
+ // Create a shelf if a user is already logged in.
+ if (shell_->session_controller()->NumberOfLoggedInUsers())
+ root_window_controller->wm_root_window_controller()->CreateShelf();
for (auto& observer : observers_)
observer.OnRootWindowControllerAdded(root_window_controller);

Powered by Google App Engine
This is Rietveld 408576698