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

Side by Side Diff: ui/app_list/signin_delegate.h

Issue 20656002: Add profile selector menu to app list. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove junk from list_model.h Created 7 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 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 UI_APP_LIST_SIGNIN_DELEGATE_H_ 5 #ifndef UI_APP_LIST_SIGNIN_DELEGATE_H_
6 #define UI_APP_LIST_SIGNIN_DELEGATE_H_ 6 #define UI_APP_LIST_SIGNIN_DELEGATE_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/observer_list.h" 9 #include "base/observer_list.h"
10 #include "base/strings/string16.h" 10 #include "base/strings/string16.h"
11 #include "ui/app_list/app_list_export.h" 11 #include "ui/app_list/app_list_export.h"
12 12
13 namespace base {
14 class FilePath;
15 }
16
13 namespace app_list { 17 namespace app_list {
14 18
15 class SigninDelegateObserver; 19 class SigninDelegateObserver;
16 20
17 class APP_LIST_EXPORT SigninDelegate { 21 class APP_LIST_EXPORT SigninDelegate {
18 public: 22 public:
19 SigninDelegate(); 23 SigninDelegate();
20 virtual ~SigninDelegate(); 24 virtual ~SigninDelegate();
21 25
22 virtual bool NeedSignin() = 0; 26 virtual bool NeedSignin() = 0;
23 virtual void ShowSignin() = 0; 27 virtual void ShowSignin() = 0;
24 virtual void OpenLearnMore() = 0; 28 virtual void OpenLearnMore() = 0;
25 virtual void OpenSettings() = 0; 29 virtual void OpenSettings() = 0;
26 30
27 virtual base::string16 GetSigninHeading() = 0; 31 virtual base::string16 GetSigninHeading() = 0;
28 virtual base::string16 GetSigninText() = 0; 32 virtual base::string16 GetSigninText() = 0;
29 virtual base::string16 GetSigninButtonText() = 0; 33 virtual base::string16 GetSigninButtonText() = 0;
30 virtual base::string16 GetLearnMoreLinkText() = 0; 34 virtual base::string16 GetLearnMoreLinkText() = 0;
31 virtual base::string16 GetSettingsLinkText() = 0; 35 virtual base::string16 GetSettingsLinkText() = 0;
32 36
37 virtual void SetProfileByPath(const base::FilePath& profile_path) = 0;
koz (OOO until 15th September) 2013/08/05 01:09:11 This should take a Profile*, seeing as implementor
benwells 2013/08/06 01:37:29 I don't think ui/app_list shouldn't depend on Prof
koz (OOO until 15th September) 2013/08/06 04:51:33 Good point. This method should probably not be in
calamity 2013/08/08 04:52:12 Done.
38
33 void AddObserver(SigninDelegateObserver* observer); 39 void AddObserver(SigninDelegateObserver* observer);
34 void RemoveObserver(SigninDelegateObserver* observer); 40 void RemoveObserver(SigninDelegateObserver* observer);
35 41
36 protected: 42 protected:
37 void NotifySigninSuccess(); 43 void NotifySigninSuccess();
38 44
39 private: 45 private:
40 ObserverList<SigninDelegateObserver, true> observers_; 46 ObserverList<SigninDelegateObserver, true> observers_;
41 47
42 DISALLOW_COPY_AND_ASSIGN(SigninDelegate); 48 DISALLOW_COPY_AND_ASSIGN(SigninDelegate);
43 }; 49 };
44 50
45 } // namespace app_list 51 } // namespace app_list
46 52
47 #endif // UI_APP_LIST_SIGNIN_DELEGATE_H_ 53 #endif // UI_APP_LIST_SIGNIN_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698