Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(32)

Unified Diff: chrome/browser/profiles/profile_manager.cc

Issue 7621031: Revert 97049 - Trying to see if it fixes sync_integration_tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/profiles/profile_manager.h ('k') | chrome/browser/ui/cocoa/browser/avatar_button.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/profiles/profile_manager.cc
===================================================================
--- chrome/browser/profiles/profile_manager.cc (revision 97074)
+++ chrome/browser/profiles/profile_manager.cc (working copy)
@@ -503,11 +503,7 @@
return profiles;
}
-ProfileInfoInterface& ProfileManager::GetProfileInfo() {
- return GetMutableProfileInfo();
-}
-
-ProfileInfoCache& ProfileManager::GetMutableProfileInfo() {
+ProfileInfoCache& ProfileManager::GetProfileInfoCache() {
if (!profile_info_cache_.get()) {
FilePath user_data_dir;
PathService::Get(chrome::DIR_USER_DATA, &user_data_dir);
@@ -518,7 +514,7 @@
}
void ProfileManager::AddProfileToCache(Profile* profile) {
- ProfileInfoCache& cache = GetMutableProfileInfo();
+ ProfileInfoCache& cache = GetProfileInfoCache();
if (profile->GetPath().DirName() != cache.GetUserDataDir())
return;
@@ -557,12 +553,13 @@
if (profile)
BrowserList::CloseAllBrowsersWithProfile(profile);
profiles_to_delete_.push_back(profile_dir);
- GetMutableProfileInfo().DeleteProfileFromCache(profile_dir);
+ ProfileInfoCache& cache = GetProfileInfoCache();
+ cache.DeleteProfileFromCache(profile_dir);
}
// static
bool ProfileManager::IsMultipleProfilesEnabled() {
-#if (defined(TOOLKIT_VIEWS) && !defined(OS_CHROMEOS)) || defined(OS_MACOSX)
+#if defined(TOOLKIT_VIEWS) && !defined(OS_CHROMEOS)
return true;
#endif
return CommandLine::ForCurrentProcess()->HasSwitch(switches::kMultiProfiles);
« no previous file with comments | « chrome/browser/profiles/profile_manager.h ('k') | chrome/browser/ui/cocoa/browser/avatar_button.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698