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

Unified Diff: chrome/browser/ui/startup/startup_browser_creator.cc

Issue 2906103002: Post-cleanup settings reset. (Closed)
Patch Set: Rebase Created 3 years, 7 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/ui/startup/startup_browser_creator.cc
diff --git a/chrome/browser/ui/startup/startup_browser_creator.cc b/chrome/browser/ui/startup/startup_browser_creator.cc
index 994ee56a84c3b536e269df86b019f223e1604019..0d4c676d2df9b5296206cc743c470befc4b91f20 100644
--- a/chrome/browser/ui/startup/startup_browser_creator.cc
+++ b/chrome/browser/ui/startup/startup_browser_creator.cc
@@ -105,6 +105,7 @@
#if defined(OS_WIN)
#include "base/win/windows_version.h"
#include "chrome/browser/metrics/jumplist_metrics_win.h"
+#include "chrome/browser/safe_browsing/chrome_cleaner/settings_resetter_win.h"
#endif
#if BUILDFLAG(ENABLE_PRINT_PREVIEW)
@@ -382,6 +383,13 @@ bool StartupBrowserCreator::LaunchBrowser(
profile_launch_observer.Get().AddLaunched(profile);
+#if defined(OS_WIN)
+// Reset settings for the current profile if it's tagged to be reset after a
+// complete run of the Chrome Cleanup tool.
+// safe_browsing::ResetPostCleanupSettingsIfTagged(
+// {profile}, base::Bind([] {}), nullptr /* use default delegate */);
alito 2017/06/02 05:32:42 Is this supposed to be uncommented or removed?
ftirelo 2017/06/02 21:20:57 Commented this while debugging the flaky browser t
+#endif
+
#if defined(OS_CHROMEOS)
chromeos::ProfileHelper::Get()->ProfileStartup(profile, process_startup);
#endif

Powered by Google App Engine
This is Rietveld 408576698