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

Unified Diff: chrome/browser/plugin_data_remover_helper.cc

Issue 10387161: Pepper Flash settings integration: add UI for "deauthorize content licenses". (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Changes in response to Scott's suggestions Created 8 years, 7 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/plugin_data_remover_helper.cc
diff --git a/chrome/browser/plugin_data_remover_helper.cc b/chrome/browser/plugin_data_remover_helper.cc
index 36470f5f38c7bdb8c87b53b8037d01b277714505..da1828f5b84777ffbb788f176a8240c836e3f49c 100644
--- a/chrome/browser/plugin_data_remover_helper.cc
+++ b/chrome/browser/plugin_data_remover_helper.cc
@@ -4,39 +4,10 @@
#include "chrome/browser/plugin_data_remover_helper.h"
-#include <string>
-
-#include "base/bind.h"
#include "chrome/browser/plugin_prefs.h"
-#include "chrome/browser/prefs/pref_service.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/common/chrome_notification_types.h"
-#include "content/public/browser/browser_thread.h"
-#include "content/public/browser/notification_source.h"
#include "content/public/browser/plugin_data_remover.h"
-#include "content/public/browser/plugin_service.h"
#include "webkit/plugins/webplugininfo.h"
-using content::BrowserThread;
-using content::PluginService;
-
-PluginDataRemoverHelper::PluginDataRemoverHelper()
- : profile_(NULL),
- ALLOW_THIS_IN_INITIALIZER_LIST(factory_(this)) {}
-
-PluginDataRemoverHelper::~PluginDataRemoverHelper() {
-}
-
-void PluginDataRemoverHelper::Init(const char* pref_name,
- Profile* profile,
- content::NotificationObserver* observer) {
- pref_.Init(pref_name, profile->GetPrefs(), observer);
- profile_ = profile;
- registrar_.Add(this, chrome::NOTIFICATION_PLUGIN_ENABLE_STATUS_CHANGED,
- content::Source<Profile>(profile));
- StartUpdate();
-}
-
// static
bool PluginDataRemoverHelper::IsSupported(PluginPrefs* plugin_prefs) {
std::vector<webkit::WebPluginInfo> plugins;
@@ -48,30 +19,3 @@ bool PluginDataRemoverHelper::IsSupported(PluginPrefs* plugin_prefs) {
}
return false;
}
-
-void PluginDataRemoverHelper::Observe(
- int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) {
- if (type == chrome::NOTIFICATION_PLUGIN_ENABLE_STATUS_CHANGED) {
- StartUpdate();
- } else {
- NOTREACHED();
- }
-}
-
-void PluginDataRemoverHelper::StartUpdate() {
- PluginService::GetInstance()->GetPlugins(
- base::Bind(&PluginDataRemoverHelper::GotPlugins, factory_.GetWeakPtr(),
- PluginPrefs::GetForProfile(profile_)));
-}
-
-void PluginDataRemoverHelper::GotPlugins(
- scoped_refptr<PluginPrefs> plugin_prefs,
- const std::vector<webkit::WebPluginInfo>& plugins) {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- bool supported = IsSupported(plugin_prefs);
- // Set the value on the PrefService instead of through the PrefMember to
- // notify observers if it changed.
- profile_->GetPrefs()->SetBoolean(pref_.GetPrefName().c_str(), supported);
-}

Powered by Google App Engine
This is Rietveld 408576698