Commit 6c6c779b authored by James Cook's avatar James Cook Committed by Commit Bot

Update docs for ProfileManager::GetActive/GetPrimaryUserProfile

These functions are used in hundreds of locations in the code base and
there's no ongoing effort to deprecate them. New developers are
sometimes confused when they write code that must use these functions.
Delete the comments discouraging their usage.

Bug: none
Change-Id: I4d40956edc6168340a361b762eb741723d47c8d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2404949
Commit-Queue: James Cook <jamescook@chromium.org>
Commit-Queue: Stefan Kuhne <skuhne@chromium.org>
Reviewed-by: default avatarStefan Kuhne <skuhne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#805986}
parent b2f9cb37
...@@ -73,16 +73,10 @@ class ProfileManager : public content::NotificationObserver, ...@@ -73,16 +73,10 @@ class ProfileManager : public content::NotificationObserver,
// Get the profile for the user which created the current session. // Get the profile for the user which created the current session.
// Note that in case of a guest account this will return a 'suitable' profile. // Note that in case of a guest account this will return a 'suitable' profile.
// This function is temporary and will soon be moved to ash. As such avoid
// using it at all cost.
// TODO(skuhne): Move into ash's new user management function.
static Profile* GetPrimaryUserProfile(); static Profile* GetPrimaryUserProfile();
// Get the profile for the currently active user. // Get the profile for the currently active user.
// Note that in case of a guest account this will return a 'suitable' profile. // Note that in case of a guest account this will return a 'suitable' profile.
// This function is temporary and will soon be moved to ash. As such avoid
// using it at all cost.
// TODO(skuhne): Move into ash's new user management function.
static Profile* GetActiveUserProfile(); static Profile* GetActiveUserProfile();
// Load and return the initial profile for browser. On ChromeOS, this returns // Load and return the initial profile for browser. On ChromeOS, this returns
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment