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

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

Issue 14923004: [Mac] AppController needs to update its "last profile" pointer when the active profile is deleted (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix broken windows code Created 7 years, 6 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
Index: chrome/browser/profiles/profile_manager.h
diff --git a/chrome/browser/profiles/profile_manager.h b/chrome/browser/profiles/profile_manager.h
index 21f403302a6ae4f8ed09b55705dde1d8e942d6f6..68f7cd029e3dd1d54a33581f7af53dd84188cded 100644
--- a/chrome/browser/profiles/profile_manager.h
+++ b/chrome/browser/profiles/profile_manager.h
@@ -289,6 +289,9 @@ class ProfileManager : public base::NonThreadSafe,
// Returns true if the profile was added, false otherwise.
bool AddProfile(Profile* profile);
+ // Schedules the profile at the given path to be deleted on shutdown.
+ void FinishDeletingProfile(const base::FilePath& profile_dir);
+
// Registers profile with given info. Returns pointer to created ProfileInfo
// entry.
ProfileInfo* RegisterProfile(Profile* profile, bool created);
@@ -326,6 +329,17 @@ class ProfileManager : public base::NonThreadSafe,
Profile* profile,
Profile::CreateStatus status);
+ // If the |loaded_profile| has been loaded succesfully (according to |status|)
+ // and isn't already scheduled for deletion, then finishes adding
+ // |profile_to_delete_dir| to the queue of profiles to be deleted, and updates
+ // the kProfileLastUsed preference to |last_non_managed_profile|.
Alexei Svitkine (slow) 2013/06/22 14:14:09 Nit: last_non_managed_profile_path
noms 2013/06/26 15:24:45 Done.
+ void OnNewActiveProfileLoaded(
+ const base::FilePath& profile_to_delete_dir,
Alexei Svitkine (slow) 2013/06/22 14:14:09 Nit: Rename to |profile_to_delete_path| so that it
noms 2013/06/26 15:24:45 Done.
+ const base::FilePath& last_non_managed_profile_path,
+ const CreateCallback& original_callback,
+ Profile* loaded_profile,
+ Profile::CreateStatus status);
+
content::NotificationRegistrar registrar_;
// The path to the user data directory (DIR_USER_DATA).

Powered by Google App Engine
This is Rietveld 408576698