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

Side by Side Diff: chrome/browser/ui/user_manager.h

Issue 564453003: Access to Chrome via the System Tray should go through the User Manager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Refactor ShowUserManager into a new static class Created 6 years, 3 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
(Empty)
1 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_UI_USER_MANAGER_H_
6 #define CHROME_BROWSER_UI_USER_MANAGER_H_
7
8 #include "chrome/browser/profiles/profile_window.h"
9
10 namespace base {
11 class FilePath;
12 }
13
14 // Cross-platform methods for displaying the user manager.
15 class UserManager {
16 public:
17 // Shows the User Manager or re-activates an existing one, focusing the
18 // profile given by |profile_path_to_focus|. Based on the value of
Peter Kasting 2014/09/15 20:17:28 Since various callers pass an empty FilePath, you
Mike Lerman 2014/09/17 18:31:45 Done.
19 // |tutorial_mode|, a tutorial could be shown, in which case
20 // |profile_path_to_focus| is ignored. After a profile is opened, execute the
Peter Kasting 2014/09/15 20:17:28 Nit: execute -> executes
Mike Lerman 2014/09/17 18:31:46 Done.
21 // |profile_open_action|.
22 static void Show(const base::FilePath& profile_path_to_focus,
23 profiles::UserManagerTutorialMode tutorial_mode,
24 profiles::UserManagerProfileSelected profile_open_action);
25
26 // Hide the User Manager.
Peter Kasting 2014/09/15 20:17:28 Nit: Hide -> Hides
Mike Lerman 2014/09/17 18:31:46 Done.
27 static void Hide();
28
29 // Returns whether or not the User Manager is showing.
Peter Kasting 2014/09/15 20:17:27 Nit: Remove "or not"
Mike Lerman 2014/09/17 18:31:45 Done.
30 static bool IsShowing();
31 };
32
33 #endif // CHROME_BROWSER_UI_USER_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698