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

Unified Diff: ui/app_list/app_list_model.cc

Issue 20656002: Add profile selector menu to app list. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rework, fix tests Created 7 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 side-by-side diff with in-line comments
Download patch
Index: ui/app_list/app_list_model.cc
diff --git a/ui/app_list/app_list_model.cc b/ui/app_list/app_list_model.cc
index a932069d26cacb0dc6cc468edc9f8c25518ebfa5..7d23844736197a4102ccb770cdb5b3c45de66dbd 100644
--- a/ui/app_list/app_list_model.cc
+++ b/ui/app_list/app_list_model.cc
@@ -11,6 +11,12 @@
namespace app_list {
+AppListModel::User::User() : active(false) {
tapted 2013/09/13 23:12:17 nit: maybe brackets like {}, on the same line. See
calamity 2013/09/16 19:45:26 Done.
+}
+
+AppListModel::User::~User() {
+}
+
AppListModel::AppListModel()
: apps_(new Apps),
search_box_(new SearchBoxModel),
@@ -40,17 +46,11 @@ void AppListModel::SetStatus(Status status) {
OnAppListModelStatusChanged());
}
-void AppListModel::SetCurrentUser(const base::string16& current_user_name,
- const base::string16& current_user_email) {
- if (current_user_name_ == current_user_name &&
- current_user_email_ == current_user_email) {
- return;
- }
- current_user_name_ = current_user_name;
- current_user_email_ = current_user_email;
+void AppListModel::SetUsers(const Users& users) {
+ users_ = users;
FOR_EACH_OBSERVER(AppListModelObserver,
observers_,
- OnAppListModelCurrentUserChanged());
+ OnAppListModelUsersChanged());
}
void AppListModel::SetSignedIn(bool signed_in) {

Powered by Google App Engine
This is Rietveld 408576698