summaryrefslogtreecommitdiffstats
path: root/chrome/browser/views
diff options
context:
space:
mode:
authorjhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-17 19:32:10 +0000
committerjhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-02-17 19:32:10 +0000
commit6bad2e767545973222ff72d69f48a0664dc7c422 (patch)
tree1a7573dbf9b45b906ca8fd67634bc7134e6963c4 /chrome/browser/views
parent4d71f12a4b02aacd35d50c2b38d3b8d06d05f5ff (diff)
downloadchromium_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.cc34
-rw-r--r--chrome/browser/views/options/content_page_view.h4
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_;