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

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

Issue 129293002: Removing GetDefaultProfileOrOffTheRecord and GetDefaultProfile(<path>) from ProfileManager's (publi… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Changed comment Created 6 years, 11 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/chrome_browser_main.cc ('k') | chrome/browser/profiles/profile_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/profiles/profile_manager.h
diff --git a/chrome/browser/profiles/profile_manager.h b/chrome/browser/profiles/profile_manager.h
index 051f1c1d562d53537b264802dd844aa83da44d63..ee83d9d44b1489f080ea94c9e37a187875bc4f65 100644
--- a/chrome/browser/profiles/profile_manager.h
+++ b/chrome/browser/profiles/profile_manager.h
@@ -61,24 +61,15 @@ class ProfileManager : public base::NonThreadSafe,
// Note that in case of a guest account this will return a 'suitable' profile.
static Profile* GetActiveUserProfile();
- // The following DEPRECATED functions should be removed: crbug.com/83792.
-
- // DEPRECATED: DO NOT USE unless in ChromeOS.
- // Returns the default profile. This adds the profile to the
- // ProfileManager if it doesn't already exist. This method returns NULL if
+ // DEPRECATED: DO NOT USE. Function will be removed soon (crbug.com/322682).
+ // Use GetPrimaryUserProfile or GetActiveUserProfile instead.
+ // Returns the default profile. This adds the profile to the
+ // ProfileManager if it doesn't already exist. This method returns NULL if
// the profile doesn't exist and we can't create it.
// The profile used can be overridden by using --login-profile on cros.
- Profile* GetDefaultProfile(const base::FilePath& user_data_dir);
-
- // DEPRECATED: DO NOT USE unless in ChromeOS.
- // Same as instance method but provides the default user_data_dir as well.
// Note that in case of a guest account this will return a 'suitable' profile.
static Profile* GetDefaultProfile();
- // DEPRECATED: DO NOT USE unless in ChromeOS.
- // Same as GetDefaultProfile(). Do not use.
- static Profile* GetDefaultProfileOrOffTheRecord();
-
// Returns a profile for a specific profile directory within the user data
// dir. This will return an existing profile it had already been created,
// otherwise it will create and manage it.
@@ -268,6 +259,14 @@ class ProfileManager : public base::NonThreadSafe,
DISALLOW_COPY_AND_ASSIGN(ProfileInfo);
};
+ // Returns the profile of the active user and / or the off the record profile
+ // if needed. This adds the profile to the ProfileManager if it doesn't
+ // already exist. The method will return NULL if the profile doesn't exist
+ // and we can't create it.
+ // The profile used can be overridden by using --login-profile on cros.
+ Profile* GetActiveUserOrOffTheRecordProfileFromPath(
+ const base::FilePath& user_data_dir);
+
// Adds a pre-existing Profile object to the set managed by this
// ProfileManager. This ProfileManager takes ownership of the Profile.
// The Profile should not already be managed by this ProfileManager.
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/profiles/profile_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698