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

Side by Side Diff: chrome/browser/ui/ash/multi_user/multi_user_window_manager_chromeos.h

Issue 890293002: Fixing crash upon window manager creation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed Created 5 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_ 5 #ifndef CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_
6 #define CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_ 6 #define CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 enum AnimationSpeed { 65 enum AnimationSpeed {
66 ANIMATION_SPEED_NORMAL, // The normal animation speed. 66 ANIMATION_SPEED_NORMAL, // The normal animation speed.
67 ANIMATION_SPEED_FAST, // Unit test speed which test animations. 67 ANIMATION_SPEED_FAST, // Unit test speed which test animations.
68 ANIMATION_SPEED_DISABLED // Unit tests which do not require animations. 68 ANIMATION_SPEED_DISABLED // Unit tests which do not require animations.
69 }; 69 };
70 70
71 // Create the manager and use |active_user_id| as the active user. 71 // Create the manager and use |active_user_id| as the active user.
72 explicit MultiUserWindowManagerChromeOS(const std::string& active_user_id); 72 explicit MultiUserWindowManagerChromeOS(const std::string& active_user_id);
73 ~MultiUserWindowManagerChromeOS() override; 73 ~MultiUserWindowManagerChromeOS() override;
74 74
75 // Initializes the manager after its creation. Should only be called once.
76 void Init();
77
75 // MultiUserWindowManager overrides: 78 // MultiUserWindowManager overrides:
76 void SetWindowOwner( 79 void SetWindowOwner(
77 aura::Window* window, const std::string& user_id) override; 80 aura::Window* window, const std::string& user_id) override;
78 const std::string& GetWindowOwner( 81 const std::string& GetWindowOwner(
79 aura::Window* window) const override; 82 aura::Window* window) const override;
80 void ShowWindowForUser( 83 void ShowWindowForUser(
81 aura::Window* window, const std::string& user_id) override; 84 aura::Window* window, const std::string& user_id) override;
82 bool AreWindowsSharedAmongUsers() const override; 85 bool AreWindowsSharedAmongUsers() const override;
83 void GetOwnersOfVisibleWindows( 86 void GetOwnersOfVisibleWindows(
84 std::set<std::string>* user_ids) const override; 87 std::set<std::string>* user_ids) const override;
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
260 263
261 // The animation between users. 264 // The animation between users.
262 scoped_ptr<UserSwichAnimatorChromeOS> animation_; 265 scoped_ptr<UserSwichAnimatorChromeOS> animation_;
263 266
264 DISALLOW_COPY_AND_ASSIGN(MultiUserWindowManagerChromeOS); 267 DISALLOW_COPY_AND_ASSIGN(MultiUserWindowManagerChromeOS);
265 }; 268 };
266 269
267 } // namespace chrome 270 } // namespace chrome
268 271
269 #endif // CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H _ 272 #endif // CHROME_BROWSER_UI_ASH_MULTI_USER_MULTI_USER_WINDOW_MANAGER_CHROMEOS_H _
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698