Start app shim when app launched.

apps/app_shim/app_shim_host_mac.cc

 // Copyright 2013 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.
 
 #include "apps/app_shim/app_shim_host_mac.h"
 
 #include "apps/app_shim/app_shim_handler_mac.h"
 #include "apps/app_shim/app_shim_messages.h"
 #include "base/bind.h"
 #include "base/files/file_path.h"
@@ skipping 44 matching lines @@
 
 void AppShimHost::OnChannelError() {
   Close();
 }
 
 bool AppShimHost::Send(IPC::Message* message) {
   DCHECK(channel_.get());
   return channel_->Send(message);
 }
 
-void AppShimHost::OnLaunchApp(std::string profile_dir, std::string app_id) {
+void AppShimHost::OnLaunchApp(base::FilePath profile_dir,
+                              std::string app_id,
+                              bool launch_now) {
   DCHECK(CalledOnValidThread());
   DCHECK(!profile_);
   if (profile_) {
     // Only one app launch message per channel.
     Send(new AppShimMsg_LaunchApp_Done(false));
     return;
   }
 
-  profile_ = FetchProfileForDirectory(profile_dir);
+  if (!(profile_ = FetchProfileForDirectory(profile_dir))) {
+    Send(new AppShimMsg_LaunchApp_Done(false));
+    return;
+  }
+
   app_id_ = app_id;
+
   apps::AppShimHandler* handler = apps::AppShimHandler::GetForAppMode(app_id_);
-  bool success = handler && handler->OnShimLaunch(this);
+  bool success = handler && handler->OnShimLaunch(this, launch_now);
   Send(new AppShimMsg_LaunchApp_Done(success));
 }
 
 void AppShimHost::OnFocus() {
   DCHECK(CalledOnValidThread());
   apps::AppShimHandler* handler = apps::AppShimHandler::GetForAppMode(app_id_);
   if (handler)
     handler->OnShimFocus(this);
 }
 
-Profile* AppShimHost::FetchProfileForDirectory(const std::string& profile_dir) {
+Profile* AppShimHost::FetchProfileForDirectory(
+    const base::FilePath& profile_dir) {
   ProfileManager* profileManager = g_browser_process->profile_manager();
-  // Even though the name of this function is "unsafe", there's no security
-  // issue here -- the check for the profile name in the profile info cache
-  // ensures that we never access any directory that isn't a known profile.
-  base::FilePath path = base::FilePath::FromUTF8Unsafe(profile_dir);
-  path = profileManager->user_data_dir().Append(path);
+  // Check for the profile name in the profile info cache to ensure that we
+  // never access any directory that isn't a known profile.
+  base::FilePath path = profileManager->user_data_dir().Append(profile_dir);
   ProfileInfoCache& cache = profileManager->GetProfileInfoCache();
-  // This ensures that the given profile path is acutally a profile that we
-  // already know about.
   if (cache.GetIndexOfProfileWithPath(path) == std::string::npos) {
     LOG(ERROR) << "Requested directory is not a known profile '"
-               << profile_dir << "'.";
+               << profile_dir.value() << "'.";
     return NULL;
   }
   Profile* profile = profileManager->GetProfile(path);
   if (!profile) {
-    LOG(ERROR) << "Couldn't get profile for directory '" << profile_dir << "'.";
+    LOG(ERROR) << "Couldn't get profile for directory '"
+               << profile_dir.value() << "'.";
     return NULL;
   }
   return profile;
 }
 
 void AppShimHost::OnAppClosed() {
   Close();
 }
 
 void AppShimHost::Close() {
   DCHECK(CalledOnValidThread());
   delete this;
 }