Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1914)

Unified Diff: chrome/browser/gtk/clear_browsing_data_dialog_gtk.cc

Issue 524026: Adds "Delete Local Storage" option to "Clear Browsing Data" for Windows and G... (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 10 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/gtk/clear_browsing_data_dialog_gtk.h ('k') | chrome/browser/views/clear_browsing_data.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/gtk/clear_browsing_data_dialog_gtk.cc
===================================================================
--- chrome/browser/gtk/clear_browsing_data_dialog_gtk.cc (revision 36077)
+++ chrome/browser/gtk/clear_browsing_data_dialog_gtk.cc (working copy)
@@ -77,6 +77,16 @@
g_signal_connect(del_cache_checkbox_, "toggled",
G_CALLBACK(HandleOnClickedWidget), this);
+ // Local Storage checkbox.
+ del_local_storage_checkbox_ = gtk_check_button_new_with_label(
+ l10n_util::GetStringUTF8(IDS_DEL_LOCAL_STORAGE_CHKBOX).c_str());
+ gtk_box_pack_start(GTK_BOX(vbox), del_local_storage_checkbox_,
+ FALSE, FALSE, 0);
+ gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(del_local_storage_checkbox_),
+ profile_->GetPrefs()->GetBoolean(prefs::kDeleteLocalStorage));
+ g_signal_connect(del_local_storage_checkbox_, "toggled",
+ G_CALLBACK(HandleOnClickedWidget), this);
+
// Cookies checkbox.
del_cookies_checkbox_ = gtk_check_button_new_with_label(
l10n_util::GetStringUTF8(IDS_DEL_COOKIES_CHKBOX).c_str());
@@ -160,6 +170,10 @@
}
if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(del_cache_checkbox_)))
items |= BrowsingDataRemover::REMOVE_CACHE;
+ if (gtk_toggle_button_get_active(
+ GTK_TOGGLE_BUTTON(del_local_storage_checkbox_))) {
+ items |= BrowsingDataRemover::REMOVE_LOCAL_STORAGE;
+ }
// BrowsingDataRemover deletes itself when done.
remover_ = new BrowsingDataRemover(profile_,
@@ -185,6 +199,10 @@
profile_->GetPrefs()->SetBoolean(prefs::kDeleteCache,
gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget)) ?
true : false);
+ } else if (widget == del_local_storage_checkbox_) {
+ profile_->GetPrefs()->SetBoolean(prefs::kDeleteLocalStorage,
+ gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget)) ?
+ true : false);
} else if (widget == del_cookies_checkbox_) {
profile_->GetPrefs()->SetBoolean(prefs::kDeleteCookies,
gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget)) ?
« no previous file with comments | « chrome/browser/gtk/clear_browsing_data_dialog_gtk.h ('k') | chrome/browser/views/clear_browsing_data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698