diff options
author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-17 19:32:10 +0000 |
---|---|---|
committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-17 19:32:10 +0000 |
commit | 6bad2e767545973222ff72d69f48a0664dc7c422 (patch) | |
tree | 1a7573dbf9b45b906ca8fd67634bc7134e6963c4 /chrome/browser/views | |
parent | 4d71f12a4b02aacd35d50c2b38d3b8d06d05f5ff (diff) | |
download | chromium_src-6bad2e767545973222ff72d69f48a0664dc7c422.zip chromium_src-6bad2e767545973222ff72d69f48a0664dc7c422.tar.gz chromium_src-6bad2e767545973222ff72d69f48a0664dc7c422.tar.bz2 |
Linux: Implement the new AutoFill section of the Options dialog.
* Updated the text of the radio buttons.
* Added a button that opens the AutoFill dialog.
BUG=33025
TEST=manually
Patch from Thiago Farina <thiago.farina@gmail.com>
Review URL: http://codereview.chromium.org/579020
Review URL: http://codereview.chromium.org/610003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39251 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views')
-rw-r--r-- | chrome/browser/views/options/content_page_view.cc | 34 | ||||
-rw-r--r-- | chrome/browser/views/options/content_page_view.h | 4 |
2 files changed, 19 insertions, 19 deletions
diff --git a/chrome/browser/views/options/content_page_view.cc b/chrome/browser/views/options/content_page_view.cc index bc27afc..89f6fb2 100644 --- a/chrome/browser/views/options/content_page_view.cc +++ b/chrome/browser/views/options/content_page_view.cc @@ -53,8 +53,8 @@ ContentPageView::ContentPageView(Profile* profile) passwords_asktosave_radio_(NULL), passwords_neversave_radio_(NULL), change_autofill_settings_button_(NULL), - form_autofill_asktosave_radio_(NULL), - form_autofill_neversave_radio_(NULL), + form_autofill_enable_radio_(NULL), + form_autofill_disable_radio_(NULL), themes_group_(NULL), themes_reset_button_(NULL), themes_gallery_link_(NULL), @@ -96,9 +96,9 @@ void ContentPageView::ButtonPressed( profile()->GetPrefs()); } ask_to_save_passwords_.SetValue(enabled); - } else if (sender == form_autofill_asktosave_radio_ || - sender == form_autofill_neversave_radio_) { - bool enabled = form_autofill_asktosave_radio_->checked(); + } else if (sender == form_autofill_enable_radio_ || + sender == form_autofill_disable_radio_) { + bool enabled = form_autofill_enable_radio_->checked(); if (enabled) { UserMetricsRecordAction("Options_FormAutofill_Enable", profile()->GetPrefs()); @@ -223,9 +223,9 @@ void ContentPageView::NotifyPrefChanged(const std::wstring* pref_name) { } if (!pref_name || *pref_name == prefs::kFormAutofillEnabled) { if (ask_to_save_form_autofill_.GetValue()) { - form_autofill_asktosave_radio_->SetChecked(true); + form_autofill_enable_radio_->SetChecked(true); } else { - form_autofill_neversave_radio_->SetChecked(true); + form_autofill_disable_radio_->SetChecked(true); } } if (!pref_name || *pref_name == prefs::kCurrentThemeID) { @@ -310,16 +310,16 @@ void ContentPageView::InitPasswordSavingGroup() { } void ContentPageView::InitFormAutofillGroup() { - form_autofill_asktosave_radio_ = new views::RadioButton( - l10n_util::GetString(IDS_OPTIONS_AUTOFILL_SAVE), + form_autofill_enable_radio_ = new views::RadioButton( + l10n_util::GetString(IDS_OPTIONS_AUTOFILL_ENABLE), kFormAutofillRadioGroup); - form_autofill_asktosave_radio_->set_listener(this); - form_autofill_asktosave_radio_->SetMultiLine(true); - form_autofill_neversave_radio_ = new views::RadioButton( - l10n_util::GetString(IDS_OPTIONS_AUTOFILL_NEVERSAVE), + form_autofill_enable_radio_->set_listener(this); + form_autofill_enable_radio_->SetMultiLine(true); + form_autofill_disable_radio_ = new views::RadioButton( + l10n_util::GetString(IDS_OPTIONS_AUTOFILL_DISABLE), kFormAutofillRadioGroup); - form_autofill_neversave_radio_->set_listener(this); - form_autofill_neversave_radio_->SetMultiLine(true); + form_autofill_disable_radio_->set_listener(this); + form_autofill_disable_radio_->SetMultiLine(true); change_autofill_settings_button_ = new views::NativeButton( this, l10n_util::GetString(IDS_OPTIONS_AUTOFILL_SETTINGS)); @@ -344,10 +344,10 @@ void ContentPageView::InitFormAutofillGroup() { GridLayout::USE_PREF, 0, 0); layout->StartRow(0, fill_column_view_set_id); - layout->AddView(form_autofill_asktosave_radio_); + layout->AddView(form_autofill_enable_radio_); layout->AddPaddingRow(0, kRelatedControlVerticalSpacing); layout->StartRow(0, fill_column_view_set_id); - layout->AddView(form_autofill_neversave_radio_); + layout->AddView(form_autofill_disable_radio_); layout->AddPaddingRow(0, kRelatedControlVerticalSpacing); layout->StartRow(0, leading_column_view_set_id); layout->AddView(change_autofill_settings_button_); diff --git a/chrome/browser/views/options/content_page_view.h b/chrome/browser/views/options/content_page_view.h index aa08199..767ab9b 100644 --- a/chrome/browser/views/options/content_page_view.h +++ b/chrome/browser/views/options/content_page_view.h @@ -88,8 +88,8 @@ class ContentPageView : public OptionsPageView, // Controls for the Form Autofill group views::NativeButton* change_autofill_settings_button_; OptionsGroupView* form_autofill_group_; - views::RadioButton* form_autofill_asktosave_radio_; - views::RadioButton* form_autofill_neversave_radio_; + views::RadioButton* form_autofill_enable_radio_; + views::RadioButton* form_autofill_disable_radio_; // Controls for the Themes group OptionsGroupView* themes_group_; |