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

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

Issue 2698683002: Forced ephemeral profile deletion on browser removal crash fix. (Closed)
Patch Set: Fixed comment. Created 3 years, 10 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 655431231a06e50259e94a48834e103a357521ee..c7bb7ad446dc108c7604e4e91613222f38a411a8 100644
--- a/chrome/browser/profiles/profile_manager.h
+++ b/chrome/browser/profiles/profile_manager.h
@@ -202,6 +202,11 @@ class ProfileManager : public base::NonThreadSafe,
// that case the callback will be called when profile creation is complete.
void ScheduleProfileForDeletion(const base::FilePath& profile_dir,
const CreateCallback& callback);
+
+ // Schedules the forced ephemeral profile at the given path to be deleted on
Mike Lerman 2017/03/01 14:24:29 Bernhard, do Ephemeral Profiles exist on CHROME_OS
pastarmovj 2017/03/01 16:54:25 I can answer this one. On chromeos we offer a much
Mike Lerman 2017/03/01 17:44:30 Ok. Palar, can you make sure this method is also i
+ // shutdown. New profiles will not be created.
+ void ScheduleForcedEphemeralProfileForDeletion(
Mike Lerman 2017/03/01 14:24:29 Why is this method public?
palar 2017/03/01 16:51:44 Good catch, thx.
+ const base::FilePath& profile_dir);
#endif
// Autoloads profiles if they are running background apps.
« no previous file with comments | « no previous file | chrome/browser/profiles/profile_manager.cc » ('j') | chrome/browser/profiles/profile_manager.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698