Google Chrome doesn't remember settings for 'Clear browing data'. Added preferences for the clear b

chrome/browser/views/clear_browsing_data.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/views/clear_browsing_data.h"
 
 #include "chrome/app/locales/locale_settings.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/standard_layout.h"
 #include "chrome/browser/template_url_model.h"
 #include "chrome/common/l10n_util.h"
 #include "chrome/views/background.h"
 #include "chrome/views/checkbox.h"
 #include "chrome/views/label.h"
 #include "chrome/views/native_button.h"
 #include "chrome/views/throbber.h"
 #include "chrome/views/window.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/common/pref_service.h"
 #include "net/url_request/url_request_context.h"
 
 #include "generated_resources.h"
 
 // The combo box is vertically aligned to the 'time-period' label, which makes
 // the combo box look a little too close to the check box above it when we use
 // standard layout to separate them. We therefore add a little extra margin to
 // the label, giving it a little breathing space.
 static const int kExtraMarginForTimePeriodLabel = 3;
 
@@ skipping 35 matching lines @@
   status_label_.SetVisible(false);
   status_label_.SetParentOwned(false);
 
   // Regular view controls we draw by ourself. First, we add the dialog label.
   delete_all_label_ = new ChromeViews::Label(
       l10n_util::GetString(IDS_CLEAR_BROWSING_DATA_LABEL));
   AddChildView(delete_all_label_);
 
   // Add all the check-boxes.
   del_history_checkbox_ =
-      AddCheckbox(l10n_util::GetString(IDS_DEL_BROWSING_HISTORY_CHKBOX), true);
+      AddCheckbox(l10n_util::GetString(IDS_DEL_BROWSING_HISTORY_CHKBOX),
+      profile_->GetPrefs()->GetBoolean(prefs::kDeleteBrowsingHistory));
 
   del_downloads_checkbox_ =
-      AddCheckbox(l10n_util::GetString(IDS_DEL_DOWNLOAD_HISTORY_CHKBOX), true);
+      AddCheckbox(l10n_util::GetString(IDS_DEL_DOWNLOAD_HISTORY_CHKBOX),
+      profile_->GetPrefs()->GetBoolean(prefs::kDeleteDownloadHistory));
 
   del_cache_checkbox_ =
-      AddCheckbox(l10n_util::GetString(IDS_DEL_CACHE_CHKBOX), true);
+      AddCheckbox(l10n_util::GetString(IDS_DEL_CACHE_CHKBOX),
+      profile_->GetPrefs()->GetBoolean(prefs::kDeleteCache));
 
   del_cookies_checkbox_ =
-      AddCheckbox(l10n_util::GetString(IDS_DEL_COOKIES_CHKBOX), true);
+      AddCheckbox(l10n_util::GetString(IDS_DEL_COOKIES_CHKBOX),
+      profile_->GetPrefs()->GetBoolean(prefs::kDeleteCookies));
 
   del_passwords_checkbox_ =
-      AddCheckbox(l10n_util::GetString(IDS_DEL_PASSWORDS_CHKBOX), false);
+      AddCheckbox(l10n_util::GetString(IDS_DEL_PASSWORDS_CHKBOX),
+      profile_->GetPrefs()->GetBoolean(prefs::kDeletePasswords));
 
   // Add a label which appears before the combo box for the time period.
   time_period_label_ = new ChromeViews::Label(
     l10n_util::GetString(IDS_CLEAR_BROWSING_DATA_TIME_LABEL));
   AddChildView(time_period_label_);
 
   // Add the combo box showing how far back in time we want to delete.
   time_period_combobox_ = new ChromeViews::ComboBox(this);
   AddChildView(time_period_combobox_);
 }
@@ skipping 199 matching lines @@
     case 3: return l10n_util::GetString(IDS_CLEAR_DATA_EVERYTHING);
     default: NOTREACHED() << L"Missing item";
              return L"?";
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ClearBrowsingDataView, ChromeViews::ButtonListener implementation:
 
 void ClearBrowsingDataView::ButtonPressed(ChromeViews::NativeButton* sender) {
+  if (sender == del_history_checkbox_)
+    profile_->GetPrefs()->SetBoolean(prefs::kDeleteBrowsingHistory,
+        del_history_checkbox_->IsSelected() ? true : false);
+  else if (sender == del_downloads_checkbox_)
+    profile_->GetPrefs()->SetBoolean(prefs::kDeleteDownloadHistory,
+        del_downloads_checkbox_->IsSelected() ? true : false);
+  else if (sender == del_cache_checkbox_)
+    profile_->GetPrefs()->SetBoolean(prefs::kDeleteCache,
+        del_cache_checkbox_->IsSelected() ? true : false);
+  else if (sender == del_cookies_checkbox_)
+    profile_->GetPrefs()->SetBoolean(prefs::kDeleteCookies,
+        del_cookies_checkbox_->IsSelected() ? true : false);
+  else if (sender == del_passwords_checkbox_)
+    profile_->GetPrefs()->SetBoolean(prefs::kDeletePasswords,
+        del_passwords_checkbox_->IsSelected() ? true : false);
+
   // When no checkbox is checked we should not have the action button enabled.
   // This forces the button to evaluate what state they should be in.
   GetDialogClientView()->UpdateDialogButtons();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ClearBrowsingDataView, private:
 
 ChromeViews::CheckBox* ClearBrowsingDataView::AddCheckbox(
     const std::wstring& text, bool checked) {
@@ skipping 68 matching lines @@
   remover_->Remove(remove_mask);
 }
 
 void ClearBrowsingDataView::OnBrowsingDataRemoverDone() {
   // No need to remove ourselves as an observer as BrowsingDataRemover deletes
   // itself after we return.
   remover_ = NULL;
   window()->Close();
 }