diff options
author | dantasse@chromium.org <dantasse@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-05 16:29:39 +0000 |
---|---|---|
committer | dantasse@chromium.org <dantasse@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-05 16:29:39 +0000 |
commit | 3d9a7f25400663330e71f022ef924a4f3ce5e710 (patch) | |
tree | 35b769ca770134b6f165e19558ed5956bf5a5605 /chrome/browser/views/options/content_page_view.cc | |
parent | c2ed64c1ab691d0fb25bace59950a9055c166a14 (diff) | |
download | chromium_src-3d9a7f25400663330e71f022ef924a4f3ce5e710.zip chromium_src-3d9a7f25400663330e71f022ef924a4f3ce5e710.tar.gz chromium_src-3d9a7f25400663330e71f022ef924a4f3ce5e710.tar.bz2 |
Turn on autofill and prefs sync by default
BUG: 34209
TEST: behavior should be the same as if you had set the --enable-sync-preferences and --enable-sync-autofill flags.
Review URL: http://codereview.chromium.org/1551014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43618 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/options/content_page_view.cc')
-rw-r--r-- | chrome/browser/views/options/content_page_view.cc | 20 |
1 files changed, 2 insertions, 18 deletions
diff --git a/chrome/browser/views/options/content_page_view.cc b/chrome/browser/views/options/content_page_view.cc index db0ef0c..e6cf2ef 100644 --- a/chrome/browser/views/options/content_page_view.cc +++ b/chrome/browser/views/options/content_page_view.cc @@ -152,14 +152,11 @@ void ContentPageView::ButtonPressed( sync_service_->EnableForUser(); ProfileSyncService::SyncEvent(ProfileSyncService::START_FROM_OPTIONS); } - } -#if defined(OS_WIN) - else if (sender == sync_customize_button_) { + } else if (sender == sync_customize_button_) { // sync_customize_button_ should be invisible if sync is not yet set up. DCHECK(sync_service_->HasSyncSetupCompleted()); CustomizeSyncWindowView::Show(GetWindow()->GetNativeWindow(), profile()); } -#endif } void ContentPageView::LinkActivated(views::Link* source, int event_flags) { @@ -440,9 +437,7 @@ void ContentPageView::InitSyncGroup() { sync_action_link_->SetController(this); sync_start_stop_button_ = new views::NativeButton(this, std::wstring()); -#if defined(OS_WIN) sync_customize_button_ = new views::NativeButton(this, std::wstring()); -#endif using views::GridLayout; using views::ColumnSet; @@ -466,16 +461,7 @@ void ContentPageView::InitSyncGroup() { layout->AddPaddingRow(0, kRelatedControlVerticalSpacing); layout->StartRow(0, single_column_view_set_id); layout->AddView(sync_start_stop_button_); - - // TODO (dantasse) Remove this big "if" when multi-datatype sync is live. -#if defined(OS_WIN) - if (CommandLine::ForCurrentProcess()->HasSwitch( - switches::kEnableSyncPreferences) || - CommandLine::ForCurrentProcess()->HasSwitch( - switches::kEnableSyncAutofill)) { - layout->AddView(sync_customize_button_); - } -#endif + layout->AddView(sync_customize_button_); sync_group_ = new OptionsGroupView(contents, l10n_util::GetString(IDS_SYNC_OPTIONS_GROUP_NAME), std::wstring(), true); @@ -503,10 +489,8 @@ void ContentPageView::UpdateSyncControls() { sync_status_label_->SetText(status_label); sync_start_stop_button_->SetEnabled(!sync_service_->WizardIsVisible()); sync_start_stop_button_->SetLabel(button_label); -#if defined(OS_WIN) sync_customize_button_->SetLabel(customize_button_label); sync_customize_button_->SetVisible(sync_setup_completed); -#endif sync_action_link_->SetText(link_label); sync_action_link_->SetVisible(!link_label.empty()); |