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

Unified Diff: chrome/browser/resources/cleanup_tool/cleanup_browser_proxy.js

Issue 2819633003: Cleanup Tool WebUI: Add functionality to the Cleanup button (Closed)
Patch Set: Remove RegisterUserInitiatedSwReporterScan, stub in handler for now Created 3 years, 8 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/resources/cleanup_tool/cleanup_browser_proxy.js
diff --git a/chrome/browser/resources/cleanup_tool/cleanup_browser_proxy.js b/chrome/browser/resources/cleanup_tool/cleanup_browser_proxy.js
index d62dbe2f3f9bce7da044e83031bb4261576e769f..fce5764801c6f163d6b8e5c808d63ce995550f77 100644
--- a/chrome/browser/resources/cleanup_tool/cleanup_browser_proxy.js
+++ b/chrome/browser/resources/cleanup_tool/cleanup_browser_proxy.js
@@ -17,6 +17,15 @@
*/
var LastScanResult;
+/**
+ * @typedef {[
+ * wasCancelled: boolean,
+ * uwsRemoved: Array<string>,
+ * requiresReboot: boolean,
+ * ]}
tommycli 2017/04/26 16:10:33 This typedef seems odd. Did you mean {{ ... }} ? I
proberge 2017/04/26 17:28:37 Good catch! I indeed meant {{...}}. I usually rely
+ */
+var CleanupResult;
+
cr.define('cleanup', function() {
/** @interface */
function CleanupBrowserProxy() {}
@@ -32,6 +41,12 @@ cr.define('cleanup', function() {
* @return {!Promise<LastScanResult>}
*/
startScan: function() {},
+
+ /**
+ * Opens the prompt to run the Chrome Cleanup Tool.
+ * @return {!Promise<CleanupResult>}
+ */
+ startCleanup: function() {},
};
/**
@@ -50,6 +65,10 @@ cr.define('cleanup', function() {
startScan: function() {
return cr.sendWithPromise('startScan');
},
+ /** @override */
+ startCleanup: function() {
+ return cr.sendWithPromise('startCleanup');
+ },
};
return {

Powered by Google App Engine
This is Rietveld 408576698