Remember the clear browsing data combo box selection.

(Checking in for external contributer Arthur Lussos <developer0420@gmail.com>, who is already listed in AUTHORS)
Reviewed at http://codereview.chromium.org/7946

Review URL: http://codereview.chromium.org/7993

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@3978 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 17 insertions, 0 deletions

diff --git a/chrome/browser/views/clear_browsing_data.cc b/chrome/browser/views/clear_browsing_data.cc
index 0b2b6f5..c4c88f4 100644
--- a/chrome/browser/views/clear_browsing_data.cc
+++ b/chrome/browser/views/clear_browsing_data.cc
@@ -98,6 +98,9 @@ void ClearBrowsingDataView::Init() {
 
   // Add the combo box showing how far back in time we want to delete.
   time_period_combobox_ = new views::ComboBox(this);
+  time_period_combobox_->SetSelectedItem(profile_->GetPrefs()->GetInteger(
+                                         prefs::kDeleteTimePeriod));
+  time_period_combobox_->SetListener(this);
   AddChildView(time_period_combobox_);
 }
 
@@ -302,6 +305,15 @@ std::wstring ClearBrowsingDataView::GetItemAt(views::ComboBox* source,
 }
 
 ////////////////////////////////////////////////////////////////////////////////
+// ClearBrowsingDataView, views::ComboBoxListener implementation:
+
+void ClearBrowsingDataView::ItemChanged(views::ComboBox* sender,
+                                        int prev_index, int new_index) {
+  if (sender == time_period_combobox_ && prev_index != new_index)
+    profile_->GetPrefs()->SetInteger(prefs::kDeleteTimePeriod, new_index);
+}
+
+////////////////////////////////////////////////////////////////////////////////
 // ClearBrowsingDataView, views::ButtonListener implementation:
 
 void ClearBrowsingDataView::ButtonPressed(views::NativeButton* sender) {
diff --git a/chrome/browser/views/clear_browsing_data.h b/chrome/browser/views/clear_browsing_data.h
index 9b000fb..f32f31d 100644
--- a/chrome/browser/views/clear_browsing_data.h
+++ b/chrome/browser/views/clear_browsing_data.h
@@ -34,6 +34,7 @@ class ClearBrowsingDataView : public views::View,
                               public views::DialogDelegate,
                               public views::ComboBox::Model,
                               public views::NativeButton::Listener,
+                              public views::ComboBox::Listener,
                               public BrowsingDataRemover::Observer {
  public:
   explicit ClearBrowsingDataView(Profile* profile);
@@ -68,6 +69,10 @@ class ClearBrowsingDataView : public views::View,
   // Overridden from views::NativeButton::Listener:
   virtual void ButtonPressed(views::NativeButton* sender);
 
+   // Overridden from views::ComboBox::Listener:
+  virtual void ItemChanged(views::ComboBox* sender, int prev_index,
+                           int new_index);
+
  private:
   // Adds a new check-box as a child to the view.
   views::CheckBox* AddCheckbox(const std::wstring& text, bool checked);