Add profile selector menu to app list.

ui/app_list/app_list_view_delegate.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_APP_LIST_APP_LIST_VIEW_DELEGATE_H_
 #define UI_APP_LIST_APP_LIST_VIEW_DELEGATE_H_
 
 #include "base/callback_forward.h"
 #include "base/strings/string16.h"
 #include "ui/app_list/app_list_export.h"
 
+struct AvatarMenuItemModel;
+class Profile;

[tapted, 2013/07/31 05:43:53]

src/ui shouldn't care about Profile* - I don't think this belongs here. E.g.
`git grep 'Profile\*'` in src/ui turns up no results.

[calamity, 2013/08/01 08:35:45]

Done.

+
 namespace base {
 class FilePath;
 }
 
 namespace gfx {
 class ImageSkia;
 }
 
 namespace app_list {
 
 class AppListItemModel;
 class AppListModel;
 class SearchResult;
 class SigninDelegate;
 
 class APP_LIST_EXPORT AppListViewDelegate {
  public:
   // AppListView owns the delegate.
   virtual ~AppListViewDelegate() {}
 
+  virtual void SetProfile(Profile* profile) = 0;

[benwells, 2013/07/31 05:47:13]

I don't think we want to add profile here. Do you need this?

[calamity, 2013/08/01 08:35:45]

Done.

+
   // Invoked to set the model that AppListView uses.
   // Note that AppListView owns the model.
   virtual void SetModel(AppListModel* model) = 0;
 
   // Gets the SigninDelegate for the app list. Owned by the AppListViewDelegate.
   virtual SigninDelegate* GetSigninDelegate() = 0;
 
   // Gets a path to a shortcut for the given app. Returns asynchronously as the
   // shortcut may not exist yet.
   virtual void GetShortcutPathForApp(
@@ skipping 27 matching lines @@
 
   // Invoked when the app list is closing.
   virtual void ViewClosing() = 0;
 
   // Invoked when the app list's activated state changes.
   virtual void ViewActivationChanged(bool active) = 0;
 
   // 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;
 
   // Open the help UI.
   virtual void OpenHelp() = 0;
 
   // Open the feedback UI.
   virtual void OpenFeedback() = 0;
+
+  // Shows the app list for the profile at position |index| in the
+  // ProfileInfoCache.

[benwells, 2013/07/31 05:47:13]

ProfileInfoCache is a chrome concept. Having indexes into it in this API is not
right.

I guess you could just change the comment to be '.. in the menu built by the
delegate in RebuildAvatarMenuItems' but I think it would be better to just make
this a path instead of an index.

[calamity, 2013/08/01 08:35:45]

Done.

+  virtual void ShowForProfileAtIndex(size_t index) = 0;
+
+  // Rebuilds the avatar menu items from the profile info cache.

[benwells, 2013/07/31 05:47:13]

Again, profile info cache is a chrome concept and shouldn't be referred to from
here.

[calamity, 2013/08/01 08:35:45]

Done.

+  virtual void RebuildAvatarMenuItems(app_list::AppListModel* model) = 0;
 };
 
 }  // namespace app_list
 
 #endif  // UI_APP_LIST_APP_LIST_VIEW_DELEGATE_H_