GetProfileByUser deprecated and renamed to GetProfileByUserUnsafe.

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 159 matching lines @@
 bool ProfileHelper::IsOwnerProfile(Profile* profile) {
   if (!profile)
     return false;
   user_manager::User* user = ProfileHelper::Get()->GetUserByProfile(profile);
   if (!user)
     return false;
 
   return user->email() == chromeos::UserManager::Get()->GetOwnerEmail();
 }
 
-//static
+// static
 bool ProfileHelper::IsPrimaryProfile(Profile* profile) {
   if (!profile)
     return false;
   user_manager::User* user = ProfileHelper::Get()->GetUserByProfile(profile);
   if (!user)
     return false;
   return user == chromeos::UserManager::Get()->GetPrimaryUser();
 }
 
 void ProfileHelper::ProfileStartup(Profile* profile, bool process_startup) {
@@ skipping 45 matching lines @@
 
 Profile* ProfileHelper::GetProfileByUser(const user_manager::User* user) {
   // This map is non-empty only in tests.
   if (!user_to_profile_for_testing_.empty()) {
     std::map<const user_manager::User*, Profile*>::const_iterator it =
         user_to_profile_for_testing_.find(user);
     return it == user_to_profile_for_testing_.end() ? NULL : it->second;
   }
 
   Profile* profile = NULL;
   if (user->is_profile_created())

[Thiemo Nagel, 2014/08/11 14:39:54]

Nit: I think it would be more readable to turn the if condition around, then you
don't need to test profile for being non-NULL in line 251:

if (!user->is_profile_created())
    return NULL;

[dzhioev (left Google), 2014/08/12 14:56:28]

Done.

     profile = ProfileHelper::GetProfileByUserIdHash(user->username_hash());
-  else
-    profile = ProfileManager::GetActiveUserProfile();
 
   // GetActiveUserProfile() or GetProfileByUserIdHash() returns a new instance
   // of ProfileImpl(), but actually its OffTheRecordProfile() should be used.
+  if (profile && UserManager::Get()->IsLoggedInAsGuest())
+    profile = profile->GetOffTheRecordProfile();
+  return profile;

[Nikita (slow), 2014/08/11 12:29:24]

nit: Insert empty line before return.

[dzhioev (left Google), 2014/08/12 14:56:28]

Done.

+}
+
+Profile* ProfileHelper::GetProfileByUserUnsafe(const user_manager::User* user) {
+  // This map is non-empty only in tests.
+  if (!user_to_profile_for_testing_.empty()) {
+    std::map<const user_manager::User*, Profile*>::const_iterator it =
+        user_to_profile_for_testing_.find(user);
+    return it == user_to_profile_for_testing_.end() ? NULL : it->second;
+  }
+
+  Profile* profile = NULL;
+  if (user->is_profile_created()) {
+    profile = ProfileHelper::GetProfileByUserIdHash(user->username_hash());
+  } else {
+    LOG(WARNING) << "ProfileHelper::GetProfileByUserUnsafe is called when "

[Nikita (slow), 2014/08/11 12:29:24]

I don't think it's a good idea to commit this WARNING message. As we know that
GetProfileByUser() may be called by various callers this will result in log file
spam.

[Thiemo Nagel, 2014/08/11 14:39:54]

How about DLOG() instead of LOG()?

[dzhioev (left Google), 2014/08/12 14:56:28]

Now this message shoots only once on adding user into session. The CL fixing
this is pending. However this message is a strong signal that something wrong is
happening, so I want to leave it as LOG.

+                    "|user|'s profile is not created. It probably means that "
+                    "something is wrong with a calling code. Please report in "
+                    "http://crbug.com/361528 if you see this message.";
+    profile = ProfileManager::GetActiveUserProfile();
+  }
+
+  // GetActiveUserProfile() or GetProfileByUserIdHash() returns a new instance
+  // of ProfileImpl(), but actually its OffTheRecordProfile() should be used.
   if (profile && UserManager::Get()->IsLoggedInAsGuest())
     profile = profile->GetOffTheRecordProfile();
   return profile;
 }
 
 user_manager::User* ProfileHelper::GetUserByProfile(Profile* profile) {
   // This map is non-empty only in tests.
   if (enable_profile_to_user_testing || !user_list_for_testing_.empty()) {
     if (always_return_primary_user_for_testing)
       return const_cast<user_manager::User*>(
@@ skipping 102 matching lines @@
   ProfileHelper::SetProfileToUserForTestingEnabled(true);
 }
 
 void ProfileHelper::SetUserToProfileMappingForTesting(
     const user_manager::User* user,
     Profile* profile) {
   user_to_profile_for_testing_[user] = profile;
 }
 
 }  // namespace chromeos