Add profile selector menu to app list.

chrome/browser/ui/app_list/app_list_service_impl.h

Index: chrome/browser/ui/app_list/app_list_service_impl.h
diff --git a/chrome/browser/ui/app_list/app_list_service_impl.h b/chrome/browser/ui/app_list/app_list_service_impl.h
index 5f7d70f34f322297c6ae513bc38fcc7776bb5b4b..11d5c493cbd52bcc81dd856a6b8bc45271680f6f 100644
--- a/chrome/browser/ui/app_list/app_list_service_impl.h
+++ b/chrome/browser/ui/app_list/app_list_service_impl.h
@@ -15,8 +15,6 @@
 #include "chrome/browser/profiles/profile_info_cache_observer.h"
 #include "chrome/browser/profiles/profile_loader.h"
 #include "chrome/browser/ui/app_list/app_list_service.h"
-#include "content/public/browser/notification_observer.h"
-#include "content/public/browser/notification_registrar.h"
 
 namespace base {
 class FilePath;
@@ -29,13 +27,18 @@ class NotificationDetails;
 
 // Parts of the AppListService implementation shared between platforms.
 class AppListServiceImpl : public AppListService,
-                           public ProfileInfoCacheObserver,
-                           public content::NotificationObserver {
+                           public ProfileInfoCacheObserver{

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

space before {

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

Done.

  public:
   static void RecordAppListLaunch();
   static void RecordAppListAppLaunch();
   static void SendAppListStats();
 
+  // Shows the app list for the profile at the current |profile_path|. If the
+  // profile has not been loaded, this will load the profile and then show the
+  // app list.
+  virtual void Show() OVERRIDE;

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

nit: there should be a comment saying these are overloaded from AppListService.
It would be nice to keep all the functions grouped together - I'll see if I can
find the place in this patch which convinced you to make them public...

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

AppListControllerDelegateWin calls Show and SetProfilePath.

+  virtual void SetProfilePath(const base::FilePath& profile_path) OVERRIDE;
+
  protected:
   AppListServiceImpl();
   virtual ~AppListServiceImpl();
@@ -63,8 +66,6 @@ class AppListServiceImpl : public AppListService,
   // Returns the app list path configured in BrowserProcess::local_state().
   virtual base::FilePath GetProfilePath(
       const base::FilePath& user_data_dir) OVERRIDE;
-  virtual void SetProfilePath(const base::FilePath& profile_path) OVERRIDE;
-  virtual void Show() OVERRIDE;
   virtual void EnableAppList(Profile* initial_profile) OVERRIDE;
   virtual AppListControllerDelegate* CreateControllerDelegate() OVERRIDE;
 
@@ -83,11 +84,6 @@ class AppListServiceImpl : public AppListService,
   virtual void OnProfileWillBeRemoved(
       const base::FilePath& profile_path) OVERRIDE;
 
-  // content::NotificationObserver
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE;
-
   // The profile the AppList is currently displaying.
   Profile* profile_;
 
@@ -98,7 +94,6 @@ class AppListServiceImpl : public AppListService,
   int pending_profile_loads_;
 
   base::WeakPtrFactory<AppListServiceImpl> weak_factory_;
-  content::NotificationRegistrar registrar_;
 
   ProfileLoader profile_loader_;