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

Unified Diff: chrome/browser/dom_ui/options/clear_browser_data_handler.cc

Issue 5964008: Add UI to the Clear Browsing Data dialog to remove Flash LSO data. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review Created 9 years, 12 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
Index: chrome/browser/dom_ui/options/clear_browser_data_handler.cc
diff --git a/chrome/browser/dom_ui/options/clear_browser_data_handler.cc b/chrome/browser/dom_ui/options/clear_browser_data_handler.cc
index 21f21216727ca3bad24aa972c08b6fdea517163f..b1c3070b0e0a22bb9f8477ac38ca1447c28b0b94 100644
--- a/chrome/browser/dom_ui/options/clear_browser_data_handler.cc
+++ b/chrome/browser/dom_ui/options/clear_browser_data_handler.cc
@@ -8,8 +8,10 @@
#include "base/basictypes.h"
#include "base/string16.h"
#include "base/values.h"
+#include "chrome/browser/browser_process.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/notification_details.h"
#include "chrome/common/pref_names.h"
#include "grit/chromium_strings.h"
#include "grit/generated_resources.h"
@@ -24,6 +26,13 @@ ClearBrowserDataHandler::~ClearBrowserDataHandler() {
}
}
+void ClearBrowserDataHandler::Initialize() {
+ clear_plugin_lso_data_enabled_.Init(prefs::kClearPluginLSODataEnabled,
+ g_browser_process->local_state(),
+ this);
+ UpdateClearPluginLSOData();
+}
+
void ClearBrowserDataHandler::GetLocalizedValues(
DictionaryValue* localized_strings) {
DCHECK(localized_strings);
@@ -43,6 +52,8 @@ void ClearBrowserDataHandler::GetLocalizedValues(
l10n_util::GetStringUTF16(IDS_DEL_COOKIES_CHKBOX));
localized_strings->SetString("deletePasswordsCheckbox",
l10n_util::GetStringUTF16(IDS_DEL_PASSWORDS_CHKBOX));
+ localized_strings->SetString("deleteLSODataCheckbox",
+ l10n_util::GetStringUTF16(IDS_DEL_FLASH_DATA_CHKBOX));
localized_strings->SetString("deleteFormDataCheckbox",
l10n_util::GetStringUTF16(IDS_DEL_FORM_DATA_CHKBOX));
localized_strings->SetString("clearBrowsingDataCommit",
@@ -89,6 +100,24 @@ void ClearBrowserDataHandler::RegisterMessages() {
NewCallback(this, &ClearBrowserDataHandler::HandleClearBrowserData));
}
+void ClearBrowserDataHandler::Observe(NotificationType type,
+ const NotificationSource& source,
+ const NotificationDetails& details) {
+ switch (type.value) {
+ case NotificationType::PREF_CHANGED: {
+ const std::string& pref_name = *Details<std::string>(details).ptr();
+ if (pref_name == prefs::kClearPluginLSODataEnabled)
+ UpdateClearPluginLSOData();
+ else
+ OptionsPageUIHandler::Observe(type, source, details);
+ break;
+ }
+
+ default:
+ OptionsPageUIHandler::Observe(type, source, details);
+ }
+}
+
void ClearBrowserDataHandler::HandleClearBrowserData(const ListValue* value) {
Profile *profile = dom_ui_->GetProfile();
PrefService *prefs = profile->GetPrefs();
@@ -104,6 +133,8 @@ void ClearBrowserDataHandler::HandleClearBrowserData(const ListValue* value) {
remove_mask |= BrowsingDataRemover::REMOVE_COOKIES;
if (prefs->GetBoolean(prefs::kDeletePasswords))
remove_mask |= BrowsingDataRemover::REMOVE_PASSWORDS;
+ if (prefs->GetBoolean(prefs::kDeleteLSOData))
+ remove_mask |= BrowsingDataRemover::REMOVE_LSO_DATA;
if (prefs->GetBoolean(prefs::kDeleteFormData))
remove_mask |= BrowsingDataRemover::REMOVE_FORM_DATA;
@@ -121,6 +152,13 @@ void ClearBrowserDataHandler::HandleClearBrowserData(const ListValue* value) {
remover_->Remove(remove_mask);
}
+void ClearBrowserDataHandler::UpdateClearPluginLSOData() {
+ scoped_ptr<Value> enabled(
+ Value::CreateBooleanValue(clear_plugin_lso_data_enabled_.GetValue()));
+ dom_ui_->CallJavascriptFunction(
+ L"ClearBrowserDataOverlay.setClearPluginLSODataEnabled", *enabled);
+}
+
void ClearBrowserDataHandler::OnBrowsingDataRemoverDone() {
// No need to remove ourselves as an observer as BrowsingDataRemover deletes
// itself after we return.
« no previous file with comments | « chrome/browser/dom_ui/options/clear_browser_data_handler.h ('k') | chrome/browser/dom_ui/options/options_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698