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

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

Issue 11232059: Clean up copy&paste in confirmation dialogs for prefs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Nit addressed. Created 8 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
Index: chrome/browser/resources/options/options.js
diff --git a/chrome/browser/resources/options/options.js b/chrome/browser/resources/options/options.js
index 8a300e31ed651629f0cfc6059265e3401bfe678b..6c012fb321ad7a060608f31a0451abda489e7e68 100644
--- a/chrome/browser/resources/options/options.js
+++ b/chrome/browser/resources/options/options.js
@@ -9,18 +9,17 @@ var AutofillEditCreditCardOverlay = options.AutofillEditCreditCardOverlay;
var AutofillOptions = options.AutofillOptions;
var BrowserOptions = options.BrowserOptions;
var ClearBrowserDataOverlay = options.ClearBrowserDataOverlay;
+var ConfirmDialog = options.ConfirmDialog;
var ContentSettings = options.ContentSettings;
var ContentSettingsExceptionsArea =
options.contentSettings.ContentSettingsExceptionsArea;
var CookiesView = options.CookiesView;
var CookiesViewApp = options.CookiesViewApp;
-var DoNotTrackConfirmOverlay = options.DoNotTrackConfirmOverlay;
var FactoryResetOverlay = options.FactoryResetOverlay;
var FontSettings = options.FontSettings;
var HandlerOptions = options.HandlerOptions;
var HomePageOverlay = options.HomePageOverlay;
var ImportDataOverlay = options.ImportDataOverlay;
-var InstantConfirmOverlay = options.InstantConfirmOverlay;
var LanguageOptions = options.LanguageOptions;
var MediaGalleriesManager = options.MediaGalleriesManager;
var OptionsFocusManager = options.OptionsFocusManager;
@@ -31,7 +30,6 @@ var PreferredNetworks = options.PreferredNetworks;
var ManageProfileOverlay = options.ManageProfileOverlay;
var SearchEngineManager = options.SearchEngineManager;
var SearchPage = options.SearchPage;
-var SpellingConfirmOverlay = options.SpellingConfirmOverlay;
var StartupOverlay = options.StartupOverlay;
var SyncSetupOverlay = options.SyncSetupOverlay;
@@ -73,6 +71,36 @@ function load() {
OptionsPage.registerOverlay(ClearBrowserDataOverlay.getInstance(),
BrowserOptions.getInstance(),
[$('privacyClearDataButton')]);
+ OptionsPage.registerOverlay(new ConfirmDialog(
+ 'doNotTrackConfirm',
+ loadTimeData.getString('doNotTrackConfirmOverlayTabTitle'),
+ 'do-not-track-confirm-overlay',
+ $('do-not-track-confirm-ok'),
+ $('do-not-track-confirm-cancel'),
+ $('do-not-track-enabled').pref,
+ $('do-not-track-enabled').metric));
+ OptionsPage.registerOverlay(new ConfirmDialog(
+ 'instantConfirm',
+ loadTimeData.getString('instantConfirmOverlayTabTitle'),
+ 'instantConfirmOverlay',
+ $('instantConfirmOk'),
+ $('instantConfirmCancel'),
+ $('instant-enabled-control').pref,
+ $('instant-enabled-control').metric,
+ 'instant.confirm_dialog_shown'));
+ // 'spelling-enabled-control' element is only present on Chrome branded
+ // builds.
+ if ($('spelling-enabled-control')) {
+ OptionsPage.registerOverlay(new ConfirmDialog(
+ 'spellingConfirm',
+ loadTimeData.getString('spellingConfirmOverlayTabTitle'),
+ 'spelling-confirm-overlay',
+ $('spelling-confirm-ok'),
+ $('spelling-confirm-cancel'),
+ $('spelling-enabled-control').pref,
+ $('spelling-enabled-control').metric,
+ 'spellcheck.confirm_dialog_shown'));
+ }
OptionsPage.registerOverlay(ContentSettings.getInstance(),
BrowserOptions.getInstance(),
[$('privacyContentSettingsButton')]);
@@ -85,8 +113,6 @@ function load() {
OptionsPage.registerOverlay(CookiesViewApp.getInstance(),
ContentSettings.getInstance(),
[$('privacyContentSettingsButton')]);
- OptionsPage.registerOverlay(DoNotTrackConfirmOverlay.getInstance(),
- BrowserOptions.getInstance());
OptionsPage.registerOverlay(FontSettings.getInstance(),
BrowserOptions.getInstance(),
[$('fontSettingsCustomizeFontsButton')]);
@@ -100,8 +126,6 @@ function load() {
[$('change-home-page')]);
OptionsPage.registerOverlay(ImportDataOverlay.getInstance(),
BrowserOptions.getInstance());
- OptionsPage.registerOverlay(InstantConfirmOverlay.getInstance(),
- BrowserOptions.getInstance());
OptionsPage.registerOverlay(LanguageOptions.getInstance(),
BrowserOptions.getInstance(),
[$('language-button')]);
@@ -116,8 +140,6 @@ function load() {
OptionsPage.registerOverlay(SearchEngineManager.getInstance(),
BrowserOptions.getInstance(),
[$('manage-default-search-engines')]);
- OptionsPage.registerOverlay(SpellingConfirmOverlay.getInstance(),
- BrowserOptions.getInstance());
OptionsPage.registerOverlay(StartupOverlay.getInstance(),
BrowserOptions.getInstance());
OptionsPage.registerOverlay(SyncSetupOverlay.getInstance(),
« no previous file with comments | « chrome/browser/resources/options/instant_confirm_overlay.js ('k') | chrome/browser/resources/options/options_bundle.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698