Fixed app launcher crashing trying to load guest profile.

chrome/browser/ui/app_list/app_list_service_impl.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 "chrome/browser/ui/app_list/app_list_service_impl.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 188 matching lines @@
         time_taken,
         base::TimeDelta::FromSeconds(1),
         base::TimeDelta::FromMinutes(kDiscoverabilityTimeoutMinutes - 1),
         10 /* bucket_count */);
   }
   UMA_HISTOGRAM_ENUMERATION("Apps.AppListHowEnabled",
                             enable_source,
                             AppListService::ENABLE_NUM_ENABLE_SOURCES);
 }
 
+// Checks whether a profile name is valid for the app list. Returns false if the
+// name is empty, or represents a guest profile.
+bool IsValidProfileName(const std::string& profile_name) {
+  if (profile_name.empty())
+    return false;
+
+  return profile_name !=
+         base::FilePath(chrome::kGuestProfileDir).AsUTF8Unsafe();
+}
+
 }  // namespace
 
 void AppListServiceImpl::RecordAppListLaunch() {
   RecordDailyEventFrequency(prefs::kLastAppListLaunchPing,
                             prefs::kAppListLaunchCount,
                             &SendAppListLaunch);
   RecordAppListDiscoverability(local_state_, false);
   RecordAppListLastLaunch();
 }
 
@@ skipping 65 matching lines @@
 void AppListServiceImpl::Init(Profile* initial_profile) {}
 
 base::FilePath AppListServiceImpl::GetProfilePath(
     const base::FilePath& user_data_dir) {
   std::string app_list_profile;
   if (local_state_->HasPrefPath(prefs::kAppListProfile))
     app_list_profile = local_state_->GetString(prefs::kAppListProfile);
 
   // If the user has no profile preference for the app launcher, default to the
   // last browser profile used.
-  if (app_list_profile.empty() &&
+  if (!IsValidProfileName(app_list_profile) &&
       local_state_->HasPrefPath(prefs::kProfileLastUsed)) {
     app_list_profile = local_state_->GetString(prefs::kProfileLastUsed);
   }
 
   // If there is no last used profile recorded, use the initial profile.
-  if (app_list_profile.empty())
+  if (!IsValidProfileName(app_list_profile))
     app_list_profile = chrome::kInitialProfile;
 
   return user_data_dir.AppendASCII(app_list_profile);
 }
 
 void AppListServiceImpl::SetProfilePath(const base::FilePath& profile_path) {
   local_state_->SetString(
       prefs::kAppListProfile,
       profile_path.BaseName().MaybeAsASCII());
 }
@@ skipping 119 matching lines @@
   if (!base::MessageLoop::current())
     return;  // In a unit test.
 
   // Send app list usage stats after a delay.
   const int kSendUsageStatsDelay = 5;
   base::MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       base::Bind(&AppListServiceImpl::SendAppListStats),
       base::TimeDelta::FromSeconds(kSendUsageStatsDelay));
 }