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

Unified Diff: chrome/browser/resources/settings/reset_page/reset_browser_proxy.js

Issue 2411383003: md-settings: add reset request origin to reset feedback proto. (Closed)
Patch Set: Addressed dbeam@'s comments Created 4 years, 2 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
« no previous file with comments | « no previous file | chrome/browser/resources/settings/reset_page/reset_page.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/reset_page/reset_browser_proxy.js
diff --git a/chrome/browser/resources/settings/reset_page/reset_browser_proxy.js b/chrome/browser/resources/settings/reset_page/reset_browser_proxy.js
index b59f487f32590240e9bb5308a90a3380a7641d29..96817a8f483d00f4af625b77628f803857ab9b70 100644
--- a/chrome/browser/resources/settings/reset_page/reset_browser_proxy.js
+++ b/chrome/browser/resources/settings/reset_page/reset_browser_proxy.js
@@ -10,9 +10,10 @@ cr.define('settings', function() {
/**
* @param {boolean} sendSettings Whether the user gave consent to upload
* broken settings to Google for analysis.
+ * @param {string} requestOrigin The origin of the reset request.
* @return {!Promise} A promise firing once resetting has completed.
*/
- performResetProfileSettings: function(sendSettings) {},
+ performResetProfileSettings: function(sendSettings, requestOrigin) {},
/**
* A method to be called when the reset profile dialog is hidden.
@@ -64,8 +65,9 @@ cr.define('settings', function() {
ResetBrowserProxyImpl.prototype = {
/** @override */
- performResetProfileSettings: function(sendSettings) {
- return cr.sendWithPromise('performResetProfileSettings', sendSettings);
+ performResetProfileSettings: function(sendSettings, requestOrigin) {
+ return cr.sendWithPromise('performResetProfileSettings',
+ sendSettings, requestOrigin);
},
/** @override */
« no previous file with comments | « no previous file | chrome/browser/resources/settings/reset_page/reset_page.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698