Add profile selector menu to app list.

ui/app_list/app_list_view_delegate.h

Index: ui/app_list/app_list_view_delegate.h
diff --git a/ui/app_list/app_list_view_delegate.h b/ui/app_list/app_list_view_delegate.h
index bd7e3d3837f75193f4d69cffe578b1c006a48ef5..80e99185611e78d5291d5a092f6ecf820e9702d9 100644
--- a/ui/app_list/app_list_view_delegate.h
+++ b/ui/app_list/app_list_view_delegate.h
@@ -29,9 +29,12 @@ class APP_LIST_EXPORT AppListViewDelegate {
   // AppListView owns the delegate.
   virtual ~AppListViewDelegate() {}
 
-  // Invoked to set the model that AppListView uses.
+  // Sets the delegate to use the profile at |profile_path|.
+  virtual void SetProfileByPath(const base::FilePath& profile_path) = 0;
+
+  // Invoked to initialize the model that AppListView uses. This binds the given

[tapted, 2013/08/05 03:01:39]

Incomplete sentence: "This binds the given.."

[calamity, 2013/08/08 04:52:12]

Done.

   // Note that AppListView owns the model.
-  virtual void SetModel(AppListModel* model) = 0;
+  virtual void InitModel(AppListModel* model) = 0;
 
   // Gets the SigninDelegate for the app list. Owned by the AppListViewDelegate.
   virtual SigninDelegate* GetSigninDelegate() = 0;
@@ -76,12 +79,6 @@ class APP_LIST_EXPORT AppListViewDelegate {
   // Returns the icon to be displayed in the window and taskbar.
   virtual gfx::ImageSkia GetWindowIcon() = 0;
 
-  // Returns the name of the current user.
-  virtual base::string16 GetCurrentUserName() = 0;
-
-  // Returns the email of the current user.
-  virtual base::string16 GetCurrentUserEmail() = 0;
-
   // Open the settings UI.
   virtual void OpenSettings() = 0;
 
@@ -90,6 +87,13 @@ class APP_LIST_EXPORT AppListViewDelegate {
 
   // Open the feedback UI.
   virtual void OpenFeedback() = 0;
+
+  // Shows the app list for the profile specified by |profile_path|.
+  virtual void ShowForProfileByPath(const base::FilePath& profile_path) = 0;
+
+  // Sets the |model|'s current user to this delegate's profile and rebuilds the

[tapted, 2013/08/05 03:01:39]

|model| has no meaning here -- perhaps refer to it by the appropriate class name

+  // ProfileMenuItems in the given |model|.
+  virtual void UpdateModelWithCurrentProfiles() = 0;

[tapted, 2013/08/05 03:01:39]

Probably shouldn't be plural Profiles. Maybe call this
UpdateMenuWithCurrentProfile ? (or UpdateMenuModelWithCurrentProfile?).

[calamity, 2013/08/08 04:52:12]

Hmm. Plural is appropriate though because it also rebuilds the ProfileMenuItems.
How about UpdateProfileItemsInModel?

 };
 
 }  // namespace app_list