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

Unified Diff: chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_controller_win.cc

Issue 2906103002: Post-cleanup settings reset. (Closed)
Patch Set: Use base::DoNothing Created 3 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/safe_browsing/chrome_cleaner/chrome_cleaner_controller_win.cc
diff --git a/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_controller_win.cc b/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_controller_win.cc
index f070575b1c22e4614ba2f6ab8c5ed26e0db46a2b..8a5f8bbaec162f4687f6a4225fe66e97f366e71b 100644
--- a/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_controller_win.cc
+++ b/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_controller_win.cc
@@ -19,9 +19,13 @@
#include "base/task_scheduler/task_traits.h"
#include "base/threading/thread_restrictions.h"
#include "base/threading/thread_task_runner_handle.h"
+#include "chrome/browser/browser_process.h"
#include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_fetcher_win.h"
#include "chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win.h"
+#include "chrome/browser/safe_browsing/chrome_cleaner/settings_resetter_win.h"
#include "chrome/browser/safe_browsing/chrome_cleaner/srt_client_info_win.h"
#include "chrome/installer/util/scoped_token_privilege.h"
#include "components/chrome_cleaner/public/constants/constants.h"
@@ -116,6 +120,21 @@ bool ChromeCleanerControllerDelegate::IsMetricsAndCrashReportingEnabled() {
return ChromeMetricsServiceAccessor::IsMetricsAndCrashReportingEnabled();
}
+void ChromeCleanerControllerDelegate::TagForResetting(Profile* profile) {
+ if (PostCleanupSettingsResetter::IsEnabled())
+ PostCleanupSettingsResetter().TagForResetting(profile);
+}
+
+void ChromeCleanerControllerDelegate::ResetTaggedProfiles(
+ std::vector<Profile*> profiles,
+ base::OnceClosure continuation) {
+ if (PostCleanupSettingsResetter::IsEnabled()) {
+ PostCleanupSettingsResetter().ResetTaggedProfiles(
+ std::move(profiles), std::move(continuation),
+ base::MakeUnique<PostCleanupSettingsResetter::Delegate>());
+ }
+}
+
// static
ChromeCleanerController* ChromeCleanerController::GetInstance() {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
@@ -178,6 +197,7 @@ void ChromeCleanerController::Scan(
}
void ChromeCleanerController::ReplyWithUserResponse(
+ Profile* profile,
UserResponse user_response) {
DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
DCHECK(prompt_user_callback_);
@@ -191,6 +211,7 @@ void ChromeCleanerController::ReplyWithUserResponse(
case UserResponse::kAccepted:
acceptance = PromptAcceptance::ACCEPTED;
new_state = State::kCleaning;
+ delegate_->TagForResetting(profile);
break;
case UserResponse::kDenied: // Fallthrough
case UserResponse::kDismissed:
@@ -397,8 +418,11 @@ void ChromeCleanerController::OnCleanerProcessDone(
}
if (process_status.exit_code == kRebootNotRequiredExitCode) {
- idle_reason_ = IdleReason::kCleaningSucceeded;
- SetStateAndNotifyObservers(State::kIdle);
+ delegate_->ResetTaggedProfiles(
+ g_browser_process->profile_manager()->GetLoadedProfiles(),
+ base::BindOnce(&ChromeCleanerController::OnSettingsResetCompleted,
+ base::Unretained(this)));
+ ResetCleanerDataAndInvalidateWeakPtrs();
return;
}
@@ -420,4 +444,9 @@ void ChromeCleanerController::InitiateReboot() {
}
}
+void ChromeCleanerController::OnSettingsResetCompleted() {
+ idle_reason_ = IdleReason::kCleaningSucceeded;
+ SetStateAndNotifyObservers(State::kIdle);
+}
+
} // namespace safe_browsing

Powered by Google App Engine
This is Rietveld 408576698