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

Side by Side Diff: chrome/browser/resources/settings/reset_page/reset_browser_proxy.js

Issue 1911573003: MD Settings: Reset profile dialog, display settings in new tab. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits. Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 cr.define('settings', function() { 5 cr.define('settings', function() {
6 /** @interface */ 6 /** @interface */
7 function ResetBrowserProxy() {} 7 function ResetBrowserProxy() {}
8 8
9 ResetBrowserProxy.prototype = { 9 ResetBrowserProxy.prototype = {
10 /** 10 /**
11 * @param {boolean} sendSettings Whether the user gave consent to upload 11 * @param {boolean} sendSettings Whether the user gave consent to upload
12 * broken settings to Google for analysis. 12 * broken settings to Google for analysis.
tommycli 2016/04/20 23:23:18 nit (optional): Are the settings really "broken se
13 * @return {!Promise} A promise firing once resetting has completed. 13 * @return {!Promise} A promise firing once resetting has completed.
14 */ 14 */
15 performResetProfileSettings: function(sendSettings) {}, 15 performResetProfileSettings: function(sendSettings) {},
16 16
17 /** 17 /**
18 * A method to be called when the reset profile dialog is hidden. 18 * A method to be called when the reset profile dialog is hidden.
19 */ 19 */
20 onHideResetProfileDialog: function() {}, 20 onHideResetProfileDialog: function() {},
21 21
22 /** 22 /**
23 * A method to be called when the reset profile banner is hidden. 23 * A method to be called when the reset profile banner is hidden.
24 */ 24 */
25 onHideResetProfileBanner: function() {}, 25 onHideResetProfileBanner: function() {},
26 26
27 /** 27 /**
28 * A method to be called when the reset profile dialog is shown. 28 * A method to be called when the reset profile dialog is shown.
29 */ 29 */
30 onShowResetProfileDialog: function() {}, 30 onShowResetProfileDialog: function() {},
31 31
32 /** @return {!Promise<!Array<{key:string, value:string}>} */
tommycli 2016/04/20 23:23:18 nit would be good to add a comment perhaps: "Gets
dpapad 2016/04/21 00:27:10 Done.
33 getReportedSettings: function() {},
34
32 <if expr="chromeos"> 35 <if expr="chromeos">
33 /** 36 /**
34 * A method to be called when the reset powerwash dialog is shown. 37 * A method to be called when the reset powerwash dialog is shown.
35 */ 38 */
36 onPowerwashDialogShow: function() {}, 39 onPowerwashDialogShow: function() {},
37 40
38 /** 41 /**
39 * Initiates a factory reset and restarts ChromeOS. 42 * Initiates a factory reset and restarts ChromeOS.
40 */ 43 */
41 requestFactoryResetRestart: function() {}, 44 requestFactoryResetRestart: function() {},
(...skipping 21 matching lines...) Expand all
63 /** @override */ 66 /** @override */
64 onHideResetProfileBanner: function() { 67 onHideResetProfileBanner: function() {
65 chrome.send('onHideResetProfileBanner'); 68 chrome.send('onHideResetProfileBanner');
66 }, 69 },
67 70
68 /** @override */ 71 /** @override */
69 onShowResetProfileDialog: function() { 72 onShowResetProfileDialog: function() {
70 chrome.send('onShowResetProfileDialog'); 73 chrome.send('onShowResetProfileDialog');
71 }, 74 },
72 75
76 /** @override */
77 getReportedSettings: function() {
78 return cr.sendWithPromise('getReportedSettings');
79 },
80
73 <if expr="chromeos"> 81 <if expr="chromeos">
74 /** @override */ 82 /** @override */
75 onPowerwashDialogShow: function() { 83 onPowerwashDialogShow: function() {
76 chrome.send('onPowerwashDialogShow'); 84 chrome.send('onPowerwashDialogShow');
77 }, 85 },
78 86
79 /** @override */ 87 /** @override */
80 requestFactoryResetRestart: function() { 88 requestFactoryResetRestart: function() {
81 chrome.send('requestFactoryResetRestart'); 89 chrome.send('requestFactoryResetRestart');
82 }, 90 },
83 </if> 91 </if>
84 }; 92 };
85 93
86 return { 94 return {
87 ResetBrowserProxyImpl: ResetBrowserProxyImpl, 95 ResetBrowserProxyImpl: ResetBrowserProxyImpl,
88 }; 96 };
89 }); 97 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698