Enable profile switching for standalone App Launcher via the Settings App.

Enable profile switching for standalone App Launcher via the Settings App. 

This adds a "Switch user" button to the settings app which when clicked closes
the settings app and launches a new app launcher instance for the target
profile.

Before: http://i.imgur.com/Cp32FTO.png 
After: http://i.imgur.com/V1y0XBz.png 

The original LGTMs for this are in https://codereview.chromium.org/11968034 but they are TBR'd here because that CL has an invalid base URL.

BUG=166983, 169114, 167094
TEST=Launch chrome with the flag --show-app-list, then switch profiles by launching the settings app and using the "switch user" button. The app list should reappear under the new profile (it's possible to tell profiles apart from what apps are installed).

TBR=tapted@chromium.org, benwells@chromium.org, sky@chromium.org, estade@chromium.org, xiyuan@chromium.org, sail@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=180346

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=180388

[commit-bot: I haz the power, 2013-01-30 03:25:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/1

[commit-bot: I haz the power, 2013-01-30 03:25:11]

Failed to apply patch for chrome/browser/ui/app_list/app_list_view_delegate.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file chrome/browser/ui/app_list/app_list_view_delegate.cc
  Hunk #1 FAILED at 15.
  1 out of 1 hunk FAILED -- saving rejects to file
chrome/browser/ui/app_list/app_list_view_delegate.cc.rej

Patch:       chrome/browser/ui/app_list/app_list_view_delegate.cc
Index: chrome/browser/ui/app_list/app_list_view_delegate.cc
diff --git a/chrome/browser/ui/app_list/app_list_view_delegate.cc
b/chrome/browser/ui/app_list/app_list_view_delegate.cc
index
95ed009f4656ceaeef33174a24d15f44c0054897..6fae2bbdfe2d067ac4f399fd0afee69b0ea0c192
100644
--- a/chrome/browser/ui/app_list/app_list_view_delegate.cc
+++ b/chrome/browser/ui/app_list/app_list_view_delegate.cc
@@ -15,25 +15,27 @@
 #include "chrome/browser/ui/ash/app_list/app_sync_ui_state_watcher.h"
 #endif
 
-AppListViewDelegate::AppListViewDelegate(AppListControllerDelegate* controller)
-    : controller_(controller) {}
+AppListViewDelegate::AppListViewDelegate(AppListControllerDelegate* controller,
+                                         Profile* profile)
+    : controller_(controller),
+      profile_(profile) {}
 
 AppListViewDelegate::~AppListViewDelegate() {}
 
 void AppListViewDelegate::SetModel(app_list::AppListModel* model) {
   if (model) {
-    Profile* profile = ProfileManager::GetDefaultProfileOrOffTheRecord();
-    apps_builder_.reset(new AppsModelBuilder(profile,
+    apps_builder_.reset(new AppsModelBuilder(profile_,
                                              model->apps(),
                                              controller_.get()));
     apps_builder_->Build();
 
-    search_builder_.reset(new SearchBuilder(profile,
+    search_builder_.reset(new SearchBuilder(profile_,
                                             model->search_box(),
                                             model->results(),
                                             controller_.get()));
 #if defined(USE_ASH)
-    app_sync_ui_state_watcher_.reset(new AppSyncUIStateWatcher(profile,
model));
+    app_sync_ui_state_watcher_.reset(new AppSyncUIStateWatcher(profile_,
+                                                               model));
 #endif
   } else {
     apps_builder_.reset();

[commit-bot: I haz the power, 2013-01-30 04:00:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/2002

[commit-bot: I haz the power, 2013-01-30 04:25:19]

Failed to trigger a try job on win_rel
HTTP Error 400: Bad Request

[commit-bot: I haz the power, 2013-01-30 04:38:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/9002

[commit-bot: I haz the power, 2013-01-30 05:11:21]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-01-30 23:36:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/32

[commit-bot: I haz the power, 2013-01-31 01:29:14]

Sorry for I got bad news for ya.
Compile failed with a clobber build on mac_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-01-31 05:29:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/6010

[commit-bot: I haz the power, 2013-01-31 07:30:36]

Failed to trigger a try job on linux_chromeos
HTTP Error 400: Bad Request

[commit-bot: I haz the power, 2013-01-31 07:30:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/12005

[commit-bot: I haz the power, 2013-01-31 08:18:10]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-01-31 10:08:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/13039

[commit-bot: I haz the power, 2013-01-31 10:46:50]

Sorry for I got bad news for ya.
Compile failed with a clobber build on mac.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-02-01 02:09:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/5012

[commit-bot: I haz the power, 2013-02-01 04:55:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/12063

[commit-bot: I haz the power, 2013-02-03 11:36:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/12063

[commit-bot: I haz the power, 2013-02-03 12:10:59]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-02-03 21:40:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/12063

[commit-bot: I haz the power, 2013-02-03 22:13:20]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-02-03 23:20:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/34002

[commit-bot: I haz the power, 2013-02-04 02:23:13]

Failed to apply patch for
chrome/browser/ui/views/app_list/app_list_controller_win.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file chrome/browser/ui/views/app_list/app_list_controller_win.cc
  Hunk #8 FAILED at 499.
  1 out of 10 hunks FAILED -- saving rejects to file
chrome/browser/ui/views/app_list/app_list_controller_win.cc.rej

Patch:       chrome/browser/ui/views/app_list/app_list_controller_win.cc
Index: chrome/browser/ui/views/app_list/app_list_controller_win.cc
diff --git a/chrome/browser/ui/views/app_list/app_list_controller_win.cc
b/chrome/browser/ui/views/app_list/app_list_controller_win.cc
index
530dfae0b251082fe44a527ca9af32a4bcd79464..6a6cca81c68185d2269e087afc9a4b9bbd104986
100644
--- a/chrome/browser/ui/views/app_list/app_list_controller_win.cc
+++ b/chrome/browser/ui/views/app_list/app_list_controller_win.cc
@@ -7,6 +7,7 @@
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/lazy_instance.h"
+#include "base/memory/weak_ptr.h"
 #include "base/path_service.h"
 #include "base/time.h"
 #include "base/timer.h"
@@ -18,6 +19,7 @@
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/platform_util.h"
+#include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/shell_integration.h"
@@ -27,7 +29,9 @@
 #include "chrome/browser/ui/extensions/application_launch.h"
 #include "chrome/browser/ui/views/browser_dialogs.h"
 #include "chrome/common/chrome_constants.h"
+#include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/installer/util/util_constants.h"
 #include "content/public/browser/browser_thread.h"
 #include "grit/chromium_strings.h"
@@ -145,24 +149,62 @@ class AppListControllerDelegateWin : public
AppListControllerDelegate {
 
 // The AppListController class manages global resources needed for the app
 // list to operate, and controls when the app list is opened and closed.
-class AppListController {
+class AppListController : public ProfileInfoCacheObserver {
  public:
-  AppListController()
-      : current_view_(NULL),
-        can_close_app_list_(true),
-        app_list_is_showing_(false) {}
-  ~AppListController() {}
+  AppListController();
 
   void set_can_close(bool can_close) { can_close_app_list_ = can_close; }
   bool can_close() { return can_close_app_list_; }
-  void CreateAppList();
-  void ShowAppList();
+  Profile* profile() const { return profile_; }
+  bool app_list_is_showing() const { return app_list_is_showing_; }
+
+  // Creates the app list view and populates it from |profile|, but doesn't
+  // show it.  Does nothing if the view already exists.
+  void InitView(Profile* profile);
+
+  // Activates the app list at the current mouse cursor location, creating the
+  // app list if necessary.
+  void ShowAppList(Profile* profile);
+
+  // Update the profile path stored in local prefs, load it (if not already
+  // loaded), and show the app list.
+  void SetProfilePath(const FilePath& profile_file_path);
+
   void DismissAppList();
   void AppListClosing();
   void AppListActivationChanged(bool active);
   app_list::AppListView* GetView() { return current_view_; }
 
+  // TODO(koz): Split the responsibility for tracking profiles into a
+  // platform-independent class.
+  // Overidden from ProfileInfoCacheObserver.
+  void OnProfileAdded(const FilePath& profilePath) OVERRIDE {}
+  // We need to watch for profile removal to keep kAppListProfile updated.
+  void OnProfileWillBeRemoved(const FilePath& profile_path) OVERRIDE;
+  void OnProfileWasRemoved(const FilePath& profile_path,
+                           const string16& profile_name) OVERRIDE {}
+  void OnProfileNameChanged(const FilePath& profile_path,
+                            const string16& profile_name) OVERRIDE {}
+  void OnProfileAvatarChanged(const FilePath& profile_path) OVERRIDE {}
+
  private:
+  // Loads a profile asynchronously and calls OnProfileLoaded() when done.
+  void LoadProfileAsync(const FilePath& profile_file_path);
+
+  // Callback for asynchronous profile load.
+  void OnProfileLoaded(int profile_load_sequence_id,
+                       Profile* profile,
+                       Profile::CreateStatus status);
+
+  // We need to keep the browser alive while we are loading a profile as that
+  // shows intent to show the app list. These two functions track our pending
+  // profile loads and start or end browser keep alive accordingly.
+  void IncrementPendingProfileLoads();
+  void DecrementPendingProfileLoads();
+
+  // Create or recreate, and initialize |current_view_| from |profile|.
+  void PopulateViewFromProfile(Profile* profile);
+
   // Utility methods for showing the app list.
   void SnapArrowLocationToTaskbarEdge(
       const gfx::Display& display,
@@ -191,6 +233,9 @@ class AppListController {
 
   app_list::PaginationModel pagination_model_;
 
+  // The profile the AppList is currently displaying.
+  Profile* profile_;
+
   // True if the controller can close the app list.
   bool can_close_app_list_;
 
@@ -198,6 +243,14 @@ class AppListController {
   // browser process keep-alives active.
   bool app_list_is_showing_;
 
+  // Incremented to indicate that pending profile loads are no longer valid.
+  int profile_load_sequence_id_;
+
+  // How many profile loads are pending.
+  int pending_profile_loads_;
+
+  base::WeakPtrFactory<AppListController> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(AppListController);
 };
 
@@ -297,15 +350,134 @@ void AppListControllerDelegateWin::LaunchApp(
       profile, extension, NEW_FOREGROUND_TAB));
 }
 
-void AppListController::CreateAppList() {
-#if !defined(USE_AURA)
+AppListController::AppListController()
+    : current_view_(NULL),
+      profile_(NULL),
+      can_close_app_list_(true),
+      app_list_is_showing_(false),
+      profile_load_sequence_id_(0),
+      pending_profile_loads_(0),
+      weak_factory_(this) {
+  ProfileManager* profile_manager = g_browser_process->profile_manager();
+  profile_manager->GetProfileInfoCache().AddObserver(this);
+}
+
+void AppListController::OnProfileWillBeRemoved(const FilePath& profile_path) {
+  // If the profile the app list uses just got deleted, reset it to the last
+  // used profile.
+  PrefService* local_state = g_browser_process->local_state();
+  std::string app_list_last_profile = local_state->GetString(
+      prefs::kAppListProfile);
+  if (profile_path.BaseName().MaybeAsASCII() == app_list_last_profile) {
+    local_state->SetString(prefs::kAppListProfile,
+        local_state->GetString(prefs::kProfileLastUsed));
+  }
+}
+
+void AppListController::SetProfilePath(const FilePath& profile_file_path) {
+  g_browser_process->local_state()->SetString(
+      prefs::kAppListProfile,
+      profile_file_path.BaseName().MaybeAsASCII());
+  ProfileManager* profile_manager = g_browser_process->profile_manager();
+  Profile* profile = profile_manager->GetProfileByPath(profile_file_path);
+
+  if (!profile) {
+    LoadProfileAsync(profile_file_path);
+    return;
+  }
+
+  ShowAppList(profile);
+}
+
+void AppListController::LoadProfileAsync(const FilePath& profile_file_path) {
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+
+  // Invalidate any pending profile path loads.
+  profile_load_sequence_id_++;
+
+  IncrementPendingProfileLoads();
+
+  ProfileManager* profile_manager = g_browser_process->profile_manager();
+  profile_manager->CreateProfileAsync(
+      profile_file_path,
+      base::Bind(&AppListController::OnProfileLoaded,
+                 weak_factory_.GetWeakPtr(), profile_load_sequence_id_),
+      string16(), string16(), false);
+}
+
+void AppListController::OnProfileLoaded(int profile_load_sequence_id,
+                                        Profile* profile,
+                                        Profile::CreateStatus status) {
+  DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
+  switch (status) {
+    case Profile::CREATE_STATUS_CREATED:
+      break;
+    case Profile::CREATE_STATUS_INITIALIZED:
+      // Only show if there has been no other profile shown since this load
+      // started.
+      if (profile_load_sequence_id == profile_load_sequence_id_)
+        ShowAppList(profile);
+      DecrementPendingProfileLoads();
+      break;
+    case Profile::CREATE_STATUS_FAIL:
+      DecrementPendingProfileLoads();
+      break;
+  }
+
+}
+
+void AppListController::IncrementPendingProfileLoads() {
+  pending_profile_loads_++;
+  if (pending_profile_loads_ == 1)
+    chrome::StartKeepAlive();
+}
+
+void AppListController::DecrementPendingProfileLoads() {
+  pending_profile_loads_--;
+  if (pending_profile_loads_ == 0)
+    chrome::EndKeepAlive();
+}
+
+void AppListController::ShowAppList(Profile* profile) {
+  DCHECK(profile);
+
+  // Invalidate any pending profile path loads.
+  profile_load_sequence_id_++;
+
+  // Do nothing if the app list is already displaying |profile|.
+  if (app_list_is_showing_ && (profile == profile_))
+    return;
+
+  DismissAppList();
+  PopulateViewFromProfile(profile);
+
+  if (!app_list_is_showing_) {
+    app_list_is_showing_ = true;
+    chrome::StartKeepAlive();
+  }
+
+  DCHECK(current_view_ && app_list_is_showing_);
+  gfx::Point cursor = gfx::Screen::GetNativeScreen()->GetCursorScreenPoint();
+  UpdateArrowPositionAndAnchorPoint(cursor);
+  current_view_->Show();
+  current_view_->GetWidget()->Activate();
+}
+
+void AppListController::InitView(Profile* profile) {
   if (current_view_)
     return;
+  PopulateViewFromProfile(profile);
+}
 
+void AppListController::PopulateViewFromProfile(Profile* profile) {
+#if !defined(USE_AURA)
+  if (profile == profile_)
+    return;
+  profile_ = profile;
   // The controller w…
(message too large)

[commit-bot: I haz the power, 2013-02-04 08:42:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/12090062/42003

[commit-bot: I haz the power, 2013-02-04 11:55:55]

Change committed as 180388