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

Side by Side Diff: chrome/browser/ui/webui/settings/chrome_cleanup_handler.cc

Issue 2906103002: Post-cleanup settings reset. (Closed)
Patch Set: Addressed Nico's comments 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2017 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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/ui/webui/settings/chrome_cleanup_handler.h" 5 #include "chrome/browser/ui/webui/settings/chrome_cleanup_handler.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/feature_list.h" 8 #include "base/feature_list.h"
9 #include "base/synchronization/lock.h" 9 #include "base/synchronization/lock.h"
10 #include "base/values.h" 10 #include "base/values.h"
11 #include "chrome/browser/profiles/profile.h"
Peter Kasting 2017/06/15 21:29:20 Nit: Do we actually need this #include? I wouldn'
ftirelo 2017/06/15 23:36:33 Done.
11 #include "chrome/browser/safe_browsing/chrome_cleaner/srt_field_trial_win.h" 12 #include "chrome/browser/safe_browsing/chrome_cleaner/srt_field_trial_win.h"
12 #include "chrome/grit/generated_resources.h" 13 #include "chrome/grit/generated_resources.h"
13 #include "content/public/browser/web_contents.h" 14 #include "content/public/browser/web_contents.h"
14 #include "content/public/browser/web_ui.h" 15 #include "content/public/browser/web_ui.h"
15 #include "ui/base/l10n/l10n_util.h" 16 #include "ui/base/l10n/l10n_util.h"
16 17
17 using safe_browsing::ChromeCleanerController; 18 using safe_browsing::ChromeCleanerController;
18 19
19 namespace settings { 20 namespace settings {
20 21
(...skipping 25 matching lines...) Expand all
46 return "cleaning_failed"; 47 return "cleaning_failed";
47 case ChromeCleanerController::IdleReason::kCleaningSucceeded: 48 case ChromeCleanerController::IdleReason::kCleaningSucceeded:
48 return "cleaning_succeeded"; 49 return "cleaning_succeeded";
49 } 50 }
50 NOTREACHED(); 51 NOTREACHED();
51 return ""; 52 return "";
52 } 53 }
53 54
54 } // namespace 55 } // namespace
55 56
56 ChromeCleanupHandler::ChromeCleanupHandler() 57 ChromeCleanupHandler::ChromeCleanupHandler(Profile* profile)
57 : controller_(ChromeCleanerController::GetInstance()) {} 58 : controller_(ChromeCleanerController::GetInstance()), profile_(profile) {}
58 59
59 ChromeCleanupHandler::~ChromeCleanupHandler() { 60 ChromeCleanupHandler::~ChromeCleanupHandler() {
60 controller_->RemoveObserver(this); 61 controller_->RemoveObserver(this);
61 } 62 }
62 63
63 void ChromeCleanupHandler::RegisterMessages() { 64 void ChromeCleanupHandler::RegisterMessages() {
64 web_ui()->RegisterMessageCallback( 65 web_ui()->RegisterMessageCallback(
65 "dismissCleanupPage", 66 "dismissCleanupPage",
66 base::Bind(&ChromeCleanupHandler::HandleDismiss, base::Unretained(this))); 67 base::Bind(&ChromeCleanupHandler::HandleDismiss, base::Unretained(this)));
67 web_ui()->RegisterMessageCallback( 68 web_ui()->RegisterMessageCallback(
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
139 140
140 CallJavascriptFunction("cr.webUIListenerCallback", 141 CallJavascriptFunction("cr.webUIListenerCallback",
141 base::Value("chrome-cleanup-on-dismiss")); 142 base::Value("chrome-cleanup-on-dismiss"));
142 // TODO(proberge): Show a prompt to reboot the system. 143 // TODO(proberge): Show a prompt to reboot the system.
143 } 144 }
144 145
145 void ChromeCleanupHandler::HandleStartCleanup(const base::ListValue* args) { 146 void ChromeCleanupHandler::HandleStartCleanup(const base::ListValue* args) {
146 DCHECK_EQ(0U, args->GetSize()); 147 DCHECK_EQ(0U, args->GetSize());
147 148
148 controller_->ReplyWithUserResponse( 149 controller_->ReplyWithUserResponse(
149 ChromeCleanerController::UserResponse::kAccepted); 150 profile_, ChromeCleanerController::UserResponse::kAccepted);
150 } 151 }
151 152
152 } // namespace settings 153 } // namespace settings
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698