Move ProfileLoader to chrome/browser/profiles.

apps/app_shim/app_shim_host_mac.h

Index: apps/app_shim/app_shim_host_mac.h
diff --git a/apps/app_shim/app_shim_host_mac.h b/apps/app_shim/app_shim_host_mac.h
index ce5ce5b4d1861ddaeb13b4ab6b7ca565a8c58965..9444d96e40762a354c82ba429039788fc6a2fe90 100644
--- a/apps/app_shim/app_shim_host_mac.h
+++ b/apps/app_shim/app_shim_host_mac.h
@@ -11,6 +11,7 @@
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/non_thread_safe.h"
+#include "chrome/browser/ui/app_list/profile_loader.h"

[tapted, 2013/06/11 05:31:44]

Hmm. This makes something of a circular dependency. I think we need to move
profile_loader to live under /apps, or at least higher up than app_list. It
doesn't make sense for the shim host to depend on what's currently the
app_list's profile loader. profile_loader.cc has includes:

#include "chrome/browser/lifetime/application_lifetime.h"
#include "chrome/browser/profiles/profile_manager.h"

profile_manager.h is already looped in to apps/DEPS, but I think you'll get a
presubmit/upload warning for application_lifetime.h.

So then the question becomes whether chrome::{Start,End}KeepAlive is something
that things in /apps should know about, or whether it should stay under
chrome/browser.

[jackhou1, 2013/06/12 09:41:32]

Done.

 #include "ipc/ipc_listener.h"
 #include "ipc/ipc_sender.h"
 
@@ -40,9 +41,12 @@ class AppShimHost : public IPC::Listener,
   void ServeChannel(const IPC::ChannelHandle& handle);
 
  protected:
+  virtual void LoadProfile(const base::FilePath& profile_dir,
+                           base::Callback<void(Profile*)> callback);
 
-  // Used internally; virtual so they can be mocked for testing.
-  virtual Profile* FetchProfileForDirectory(const base::FilePath& profile_dir);
+  virtual void LaunchAppWithProfile(const std::string& app_id,
+                                    apps::AppShimLaunchType launch_type,
+                                    Profile* profile);
 
   // IPC::Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
@@ -79,6 +83,8 @@ class AppShimHost : public IPC::Listener,
   scoped_ptr<IPC::ChannelProxy> channel_;
   std::string app_id_;
   Profile* profile_;
+  ProfileLoader profile_loader_;
+  base::WeakPtrFactory<AppShimHost> weak_factory_;
 };
 
 #endif  // CHROME_BROWSER_WEB_APPLICATIONS_APP_SHIM_HOST_MAC_H_