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

Side by Side Diff: chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_dialog_controller_impl_win.cc

Issue 2966453002: Chrome Cleaner UI: Add logs upload permission checkbox to the dialog (Closed)
Patch Set: More comments Created 3 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_dialog_cont roller_impl_win.h" 5 #include "chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_dialog_cont roller_impl_win.h"
6 6
7 #include "chrome/browser/ui/browser.h" 7 #include "chrome/browser/ui/browser.h"
8 #include "chrome/browser/ui/browser_dialogs.h" 8 #include "chrome/browser/ui/browser_dialogs.h"
9 #include "chrome/browser/ui/browser_finder.h" 9 #include "chrome/browser/ui/browser_finder.h"
10 #include "chrome/browser/ui/browser_list.h" 10 #include "chrome/browser/ui/browser_list.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 cleaner_controller_->state()); 50 cleaner_controller_->state());
51 51
52 cleaner_controller_->AddObserver(this); 52 cleaner_controller_->AddObserver(this);
53 } 53 }
54 54
55 ChromeCleanerDialogControllerImpl::~ChromeCleanerDialogControllerImpl() = 55 ChromeCleanerDialogControllerImpl::~ChromeCleanerDialogControllerImpl() =
56 default; 56 default;
57 57
58 void ChromeCleanerDialogControllerImpl::DialogShown() {} 58 void ChromeCleanerDialogControllerImpl::DialogShown() {}
59 59
60 void ChromeCleanerDialogControllerImpl::Accept() { 60 void ChromeCleanerDialogControllerImpl::Accept(bool logs_enabled) {
61 DCHECK(browser_); 61 DCHECK(browser_);
62 62
63 cleaner_controller_->ReplyWithUserResponse( 63 cleaner_controller_->ReplyWithUserResponse(
64 browser_->profile(), ChromeCleanerController::UserResponse::kAccepted); 64 browser_->profile(),
65 logs_enabled
66 ? ChromeCleanerController::UserResponse::kAcceptedWithLogs
67 : ChromeCleanerController::UserResponse::kAcceptedWithoutLogs);
65 OpenSettingsPage(browser_); 68 OpenSettingsPage(browser_);
66 OnInteractionDone(); 69 OnInteractionDone();
67 } 70 }
68 71
69 void ChromeCleanerDialogControllerImpl::Cancel() { 72 void ChromeCleanerDialogControllerImpl::Cancel() {
70 DCHECK(browser_); 73 DCHECK(browser_);
71 74
72 cleaner_controller_->ReplyWithUserResponse( 75 cleaner_controller_->ReplyWithUserResponse(
73 browser_->profile(), ChromeCleanerController::UserResponse::kDenied); 76 browser_->profile(), ChromeCleanerController::UserResponse::kDenied);
74 OnInteractionDone(); 77 OnInteractionDone();
75 } 78 }
76 79
77 void ChromeCleanerDialogControllerImpl::Close() { 80 void ChromeCleanerDialogControllerImpl::Close() {
78 DCHECK(browser_); 81 DCHECK(browser_);
79 82
80 cleaner_controller_->ReplyWithUserResponse( 83 cleaner_controller_->ReplyWithUserResponse(
81 browser_->profile(), ChromeCleanerController::UserResponse::kDismissed); 84 browser_->profile(), ChromeCleanerController::UserResponse::kDismissed);
82 OnInteractionDone(); 85 OnInteractionDone();
83 } 86 }
84 87
85 void ChromeCleanerDialogControllerImpl::DetailsButtonClicked() { 88 void ChromeCleanerDialogControllerImpl::DetailsButtonClicked(
89 bool logs_enabled) {
90 cleaner_controller_->SetLogsEnabled(logs_enabled);
86 OpenSettingsPage(browser_); 91 OpenSettingsPage(browser_);
87 OnInteractionDone(); 92 OnInteractionDone();
88 } 93 }
89 94
95 void ChromeCleanerDialogControllerImpl::SetLogsEnabled(bool logs_enabled) {
96 cleaner_controller_->SetLogsEnabled(logs_enabled);
97 }
98
99 bool ChromeCleanerDialogControllerImpl::LogsEnabled() {
100 return cleaner_controller_->logs_enabled();
101 }
102
90 void ChromeCleanerDialogControllerImpl::OnIdle( 103 void ChromeCleanerDialogControllerImpl::OnIdle(
91 ChromeCleanerController::IdleReason idle_reason) { 104 ChromeCleanerController::IdleReason idle_reason) {
92 if (!dialog_shown_) 105 if (!dialog_shown_)
93 OnInteractionDone(); 106 OnInteractionDone();
94 } 107 }
95 108
96 void ChromeCleanerDialogControllerImpl::OnScanning() { 109 void ChromeCleanerDialogControllerImpl::OnScanning() {
97 // This notification is received when the object is first added as an observer 110 // This notification is received when the object is first added as an observer
98 // of cleaner_controller_. 111 // of cleaner_controller_.
99 // 112 //
(...skipping 29 matching lines...) Expand all
129 if (!dialog_shown_) 142 if (!dialog_shown_)
130 OnInteractionDone(); 143 OnInteractionDone();
131 } 144 }
132 145
133 void ChromeCleanerDialogControllerImpl::OnInteractionDone() { 146 void ChromeCleanerDialogControllerImpl::OnInteractionDone() {
134 cleaner_controller_->RemoveObserver(this); 147 cleaner_controller_->RemoveObserver(this);
135 delete this; 148 delete this;
136 } 149 }
137 150
138 } // namespace safe_browsing 151 } // namespace safe_browsing
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698