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

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

Issue 2966453002: Chrome Cleaner UI: Add logs upload permission checkbox to the dialog (Closed)
Patch Set: Rebase 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 9eb9da33abb64205e1e5f25fb01353a712534561..d0ea9bece2d2f8fbf936c912417564a6f5df1b2f 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
@@ -157,6 +157,15 @@ bool ChromeCleanerController::ShouldShowCleanupInSettingsUI() {
state == State::kRebootRequired;
}
+void ChromeCleanerController::SetLogsEnabled(bool logs_enabled) {
+ if (logs_enabled_ == logs_enabled)
+ return;
+
+ logs_enabled_ = logs_enabled;
+ for (auto& observer : observer_list_)
+ observer.OnLogsEnabledChanged(logs_enabled_);
+}
+
void ChromeCleanerController::SetDelegateForTesting(
ChromeCleanerControllerDelegate* delegate) {
DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
@@ -214,8 +223,15 @@ void ChromeCleanerController::ReplyWithUserResponse(
PromptAcceptance acceptance = PromptAcceptance::DENIED;
State new_state = State::kIdle;
switch (user_response) {
- case UserResponse::kAccepted:
- acceptance = PromptAcceptance::ACCEPTED;
+ case UserResponse::kAcceptedWithLogs:
+ acceptance = PromptAcceptance::ACCEPTED_WITH_LOGS;
+ SetLogsEnabled(true);
+ new_state = State::kCleaning;
+ delegate_->TagForResetting(profile);
+ break;
+ case UserResponse::kAcceptedWithoutLogs:
+ acceptance = PromptAcceptance::ACCEPTED_WITHOUT_LOGS;
+ SetLogsEnabled(false);
new_state = State::kCleaning;
delegate_->TagForResetting(profile);
break;

Powered by Google App Engine
This is Rietveld 408576698