From 05dfe6a91f4670bfc442b3ab918e5092b3eff6cd Mon Sep 17 00:00:00 2001 From: "jeremy@chromium.org" Date: Tue, 9 Mar 2010 14:18:09 +0000 Subject: Add enable/disable translate pref to "Under the Hood". NIB change: Added checkbox for pref into web content section of "Under the Hood" and moved other controls down to match. BUG=37527 TEST=navigate to www.google.co.il using a branded build translate infobar should appear/not appear based on pref. Review URL: http://codereview.chromium.org/722004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41026 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/app/nibs/Preferences.xib | 114 ++++++++++++++++----- .../browser/cocoa/preferences_window_controller.h | 1 + .../browser/cocoa/preferences_window_controller.mm | 18 ++++ 3 files changed, 108 insertions(+), 25 deletions(-) diff --git a/chrome/app/nibs/Preferences.xib b/chrome/app/nibs/Preferences.xib index 9800b61..d4877a7 100644 --- a/chrome/app/nibs/Preferences.xib +++ b/chrome/app/nibs/Preferences.xib @@ -3,14 +3,14 @@ 1050 9L31a - 680 + 677 949.54 353.00 YES - + YES @@ -165,7 +165,7 @@ 265 - {{356, 297}, {73, 32}} + {{356, 317}, {73, 32}} YES @@ -189,7 +189,7 @@ 268 - {{17, 586}, {447, 17}} + {{17, 606}, {447, 17}} YES @@ -226,7 +226,7 @@ 266 - {{17, 172}, {409, 17}} + {{17, 192}, {409, 17}} YES @@ -242,7 +242,7 @@ 266 - {{29, 242}, {397, 17}} + {{29, 262}, {397, 17}} YES @@ -262,7 +262,7 @@ 266 - {{29, 509}, {397, 17}} + {{29, 529}, {397, 17}} YES @@ -278,7 +278,7 @@ 268 - {{38, 201}, {158, 32}} + {{38, 221}, {158, 32}} YES @@ -298,7 +298,7 @@ 266 - {{30, 449}, {395, 18}} + {{30, 469}, {395, 18}} YES @@ -325,7 +325,7 @@ 266 - {{30, 429}, {395, 18}} + {{30, 449}, {395, 18}} YES @@ -347,7 +347,7 @@ 266 - {{30, 409}, {395, 18}} + {{30, 429}, {395, 18}} YES @@ -405,7 +405,7 @@ 266 - {{30, 389}, {395, 18}} + {{30, 409}, {395, 18}} YES @@ -465,7 +465,7 @@ 268 - {{17, 334}, {447, 17}} + {{17, 354}, {447, 17}} YES @@ -490,7 +490,7 @@ NSFilenamesPboardType - {{25, 307}, {329, 17}} + {{25, 327}, {329, 17}} YES @@ -511,7 +511,7 @@ 266 - {{42, 279}, {383, 18}} + {{42, 299}, {383, 18}} YES @@ -553,7 +553,7 @@ 266 - {{30, 369}, {395, 18}} + {{30, 389}, {395, 18}} YES @@ -575,7 +575,7 @@ 268 - {{29, 487}, {96, 16}} + {{29, 507}, {96, 16}} YES @@ -595,7 +595,7 @@ 268 - {{228, 538}, {150, 32}} + {{228, 558}, {150, 32}} YES @@ -615,7 +615,7 @@ 268 - {{26, 538}, {202, 32}} + {{26, 558}, {202, 32}} YES @@ -632,8 +632,30 @@ 25 + + + 266 + {{30, 168}, {383, 18}} + + YES + + 67239424 + 0 + ^IDS_OPTIONS_TRANSLATE_ENABLE_TRANSLATE + + + 1211912703 + 130 + + + + + 200 + 25 + + - {443, 623} + {443, 643} NSView @@ -3594,6 +3616,22 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA 641 + + + value: translateEnabled + + + + + + value: translateEnabled + value + translateEnabled + 2 + + + 646 + @@ -3662,13 +3700,14 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA - + + + - - + Under the Hood Content @@ -4774,6 +4813,20 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA + + 642 + + + YES + + + + + + 643 + + + @@ -4938,6 +4991,8 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA 634.IBPluginDependency 635.IBPluginDependency 64.IBPluginDependency + 642.IBPluginDependency + 643.IBPluginDependency 65.IBPluginDependency 66.IBPluginDependency 67.IBPluginDependency @@ -5131,7 +5186,9 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA com.apple.InterfaceBuilder.CocoaPlugin com.apple.InterfaceBuilder.CocoaPlugin com.apple.InterfaceBuilder.CocoaPlugin - {{367, 295}, {443, 623}} + com.apple.InterfaceBuilder.CocoaPlugin + com.apple.InterfaceBuilder.CocoaPlugin + {{432, 838}, {443, 643}} com.apple.InterfaceBuilder.CocoaPlugin com.apple.InterfaceBuilder.CocoaPlugin com.apple.InterfaceBuilder.CocoaPlugin @@ -5171,7 +5228,7 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA - 641 + 646 @@ -5307,6 +5364,13 @@ AAMAAAABAAEAAAFTAAMAAAAEAAAFwgAAAAAACAAIAAgACAABAAEAAQABA + NSWindow + + IBProjectSource + browser/cocoa/themed_window.h + + + PreferencesWindowController NSWindowController diff --git a/chrome/browser/cocoa/preferences_window_controller.h b/chrome/browser/cocoa/preferences_window_controller.h index 1768fce..b064d87 100644 --- a/chrome/browser/cocoa/preferences_window_controller.h +++ b/chrome/browser/cocoa/preferences_window_controller.h @@ -105,6 +105,7 @@ class ProfileSyncService; IBOutlet NSButton* downloadLocationButton_; StringPrefMember defaultDownloadLocation_; BooleanPrefMember askForSaveLocation_; + BooleanPrefMember translateEnabled_; FontLanguageSettingsController* fontLanguageSettings_; StringPrefMember currentTheme_; IBOutlet NSButton* enableLoggingCheckbox_; diff --git a/chrome/browser/cocoa/preferences_window_controller.mm b/chrome/browser/cocoa/preferences_window_controller.mm index f380661..5cce358 100644 --- a/chrome/browser/cocoa/preferences_window_controller.mm +++ b/chrome/browser/cocoa/preferences_window_controller.mm @@ -390,6 +390,7 @@ CGFloat AutoSizeUnderTheHoodContent(NSView* view, - (void)setSafeBrowsing:(BOOL)value; - (void)setMetricsRecording:(BOOL)value; - (void)setAskForSaveLocation:(BOOL)value; +- (void)setTranslateEnabled:(BOOL)value; - (void)displayPreferenceViewForPage:(OptionsPage)page animate:(BOOL)animate; @end @@ -818,6 +819,7 @@ void PersonalDataManagerObserver::ShowAutoFillDialog( useSuggest_.Init(prefs::kSearchSuggestEnabled, prefs_, observer_.get()); dnsPrefetch_.Init(prefs::kDnsPrefetchingEnabled, prefs_, observer_.get()); safeBrowsing_.Init(prefs::kSafeBrowsingEnabled, prefs_, observer_.get()); + translateEnabled_.Init(prefs::kEnableTranslate, prefs_, observer_.get()); // During unit tests, there is no local state object, so we fall back to // the prefs object (where we've explicitly registered this pref so we @@ -1426,6 +1428,9 @@ const int kDisabledIndex = 1; else if (*prefName == prefs::kPromptForDownload) { [self setAskForSaveLocation:askForSaveLocation_.GetValue() ? YES : NO]; } + else if (*prefName == prefs::kEnableTranslate) { + [self setTranslateEnabled:translateEnabled_.GetValue() ? YES : NO]; + } } // Set the new download path and notify the UI via KVO. @@ -1609,6 +1614,19 @@ const int kDisabledIndex = 1; askForSaveLocation_.SetValue(value); } +- (BOOL)translateEnabled { + return translateEnabled_.GetValue(); +} + +- (void)setTranslateEnabled:(BOOL)value { + if (value) { + [self recordUserAction:"Options_Translate_Enable"]; + } else { + [self recordUserAction:"Options_Translate_Disable"]; + } + translateEnabled_.SetValue(value); +} + - (void)fontAndLanguageEndSheet:(NSWindow*)sheet returnCode:(NSInteger)returnCode contextInfo:(void*)context { -- cgit v1.1