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

Unified Diff: chrome/browser/resources/options/automatic_settings_reset_banner.js

Issue 410293004: Split OptionsPage into Page and PageManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ugh just no Created 6 years, 5 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/options/automatic_settings_reset_banner.js
diff --git a/chrome/browser/resources/options/automatic_settings_reset_banner.js b/chrome/browser/resources/options/automatic_settings_reset_banner.js
index 09439f53339359a1f4ab258be13b82ca83371e24..2a1573eb3f3b14f8b038effde6896c2768dd7047 100644
--- a/chrome/browser/resources/options/automatic_settings_reset_banner.js
+++ b/chrome/browser/resources/options/automatic_settings_reset_banner.js
@@ -6,6 +6,7 @@
cr.define('options', function() {
/** @const */ var SettingsBannerBase = options.SettingsBannerBase;
+ /** @const */ var PageManager = cr.ui.pageManager.PageManager;
/**
* AutomaticSettingsResetBanner class
@@ -43,7 +44,7 @@ cr.define('options', function() {
function(event) {
chrome.send('metricsHandler:recordAction',
['AutomaticSettingsReset_WebUIBanner_ResetClicked']);
- OptionsPage.navigateToPage('resetProfileSettings');
+ PageManager.showPageByName('resetProfileSettings');
};
},
};
« no previous file with comments | « chrome/browser/resources/options/autofill_options.js ('k') | chrome/browser/resources/options/browser_options.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698