Always load signin profile on Chrome OS startup.

chrome/browser/chromeos/profiles/profile_helper.cc

 // Copyright (c) 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/chromeos/profiles/profile_helper.h"
 
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browsing_data/browsing_data_helper.h"
@@ skipping 123 matching lines @@
 base::FilePath ProfileHelper::GetUserProfileDir(
     const std::string& user_id_hash) {
   CHECK(!user_id_hash.empty());
   return ShouldAddProfileDirPrefix(user_id_hash)
              ? base::FilePath(chrome::kProfileDirPrefix + user_id_hash)
              : base::FilePath(user_id_hash);
 }
 
 // static
 bool ProfileHelper::IsSigninProfile(const Profile* profile) {
-  return profile->GetPath().BaseName().value() == chrome::kInitialProfile;
+  return profile &&

[mtomasz, 2015/02/26 04:27:50]

nit: Passing null doesn't make sense. Can we add a DCHECK that |profile| is not
null instead and add a comment to IsSigninProfile() that |profile| must not be
null? AFAIK this is what we do usually.

[Ivan Podogov, 2015/02/26 11:02:08]

I looked at the functions above and below this one, and somehow it doesn't seem
like this is what's usually done. :)
I'll keep it consistent with the rest of this file for now.

+         profile->GetPath().BaseName().value() == chrome::kInitialProfile;
 }
 
 // static
 bool ProfileHelper::IsOwnerProfile(Profile* profile) {
   if (!profile)
     return false;
   const user_manager::User* user =
       ProfileHelper::Get()->GetUserByProfile(profile);
   if (!user)
     return false;
@@ skipping 227 matching lines @@
   user_to_profile_for_testing_[user] = profile;
 }
 
 // static
 std::string ProfileHelper::GetUserIdHashByUserIdForTesting(
     const std::string& user_id) {
   return user_id + kUserIdHashSuffix;
 }
 
 }  // namespace chromeos