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

Unified Diff: chrome/test/data/webui/settings/reset_page_test.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 | « chrome/browser/ui/webui/settings/reset_settings_handler_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/settings/reset_page_test.js
diff --git a/chrome/test/data/webui/settings/reset_page_test.js b/chrome/test/data/webui/settings/reset_page_test.js
index 9fa0cb1232184d4b20ae96ba065e0103276dcc87..f5ebbdf9243205fece1dde35cc0d61ded89993ce 100644
--- a/chrome/test/data/webui/settings/reset_page_test.js
+++ b/chrome/test/data/webui/settings/reset_page_test.js
@@ -11,6 +11,10 @@ cr.define('settings_reset_page', function() {
ResetBannerReset: 'ResetBannerReset',
ResetProfileDialogAction: 'ResetProfileDialogAction',
ResetProfileDialogOpenClose: 'ResetProfileDialogOpenClose',
+ ResetProfileDialogOriginUnknown: 'ResetProfileDialogOriginUnknown',
+ ResetProfileDialogOriginUserClick: 'ResetProfileDialogOriginUserClick',
+ ResetProfileDialogOriginTriggeredReset:
+ 'ResetProfileDialogOriginTriggeredReset',
};
/**
@@ -25,6 +29,7 @@ cr.define('settings_reset_page', function() {
'onHideResetProfileBanner',
'onShowResetProfileDialog',
'showReportedSettings',
+ 'getTriggeredResetToolName',
'onPowerwashDialogShow',
]);
};
@@ -33,8 +38,8 @@ cr.define('settings_reset_page', function() {
__proto__: settings.TestBrowserProxy.prototype,
/** @override */
- performResetProfileSettings: function(sendSettings) {
- this.methodCalled('performResetProfileSettings');
+ performResetProfileSettings: function(sendSettings, requestOrigin) {
+ this.methodCalled('performResetProfileSettings', requestOrigin);
return Promise.resolve();
},
@@ -59,6 +64,12 @@ cr.define('settings_reset_page', function() {
},
/** @override */
+ getTriggeredResetToolName: function() {
+ this.methodCalled('getTriggeredResetToolName');
+ return Promise.resolve('WonderfulAV');
+ },
+
+ /** @override */
onPowerwashDialogShow: function() {
this.methodCalled('onPowerwashDialogShow');
},
@@ -220,6 +231,36 @@ cr.define('settings_reset_page', function() {
});
});
+ function testResetRequestOrigin(expectedOrigin) {
+ var dialog = resetPage.$$('settings-reset-profile-dialog');
+ assertTrue(!!dialog);
+ MockInteractions.tap(dialog.$.reset);
+ return resetPageBrowserProxy.whenCalled(
+ 'performResetProfileSettings').then(function(resetRequest) {
+ assertEquals(expectedOrigin, resetRequest);
+ });
+ }
+
+ test(TestNames.ResetProfileDialogOriginUnknown, function() {
+ settings.navigateTo(settings.Route.RESET_DIALOG);
+ return resetPageBrowserProxy.whenCalled('onShowResetProfileDialog')
+ .then(function() { return testResetRequestOrigin(''); });
+ });
+
+ test(TestNames.ResetProfileDialogOriginUserClick, function() {
+ MockInteractions.tap(resetPage.$.resetProfile);
+ return resetPageBrowserProxy.whenCalled('onShowResetProfileDialog')
+ .then(function() { return testResetRequestOrigin('userclick'); });
+ });
+
+ test(TestNames.ResetProfileDialogOriginTriggeredReset, function() {
+ settings.navigateTo(settings.Route.TRIGGERED_RESET_DIALOG);
+ return resetPageBrowserProxy.whenCalled('onShowResetProfileDialog')
+ .then(function() {
+ return testResetRequestOrigin('triggeredreset');
+ });
+ });
+
if (cr.isChromeOS) {
/**
* @param {function(SettingsPowerwashDialogElemeent):!Element}
« no previous file with comments | « chrome/browser/ui/webui/settings/reset_settings_handler_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698