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

Unified Diff: chrome/browser/plugin_data_remover_helper.cc

Issue 10351013: Return a list of plugins that support clearing site data instead of the first one. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: comment Created 8 years, 8 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 | « no previous file | content/browser/plugin_data_remover_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 558140c09b425c15b43bb7cd15bb3ab0f636455a..36470f5f38c7bdb8c87b53b8037d01b277714505 100644
--- a/chrome/browser/plugin_data_remover_helper.cc
+++ b/chrome/browser/plugin_data_remover_helper.cc
@@ -39,9 +39,14 @@ void PluginDataRemoverHelper::Init(const char* pref_name,
// static
bool PluginDataRemoverHelper::IsSupported(PluginPrefs* plugin_prefs) {
- webkit::WebPluginInfo plugin;
- return content::PluginDataRemover::IsSupported(&plugin) &&
- plugin_prefs->IsPluginEnabled(plugin);
+ std::vector<webkit::WebPluginInfo> plugins;
+ content::PluginDataRemover::GetSupportedPlugins(&plugins);
+ for (std::vector<webkit::WebPluginInfo>::const_iterator it = plugins.begin();
+ it != plugins.end(); ++it) {
+ if (plugin_prefs->IsPluginEnabled(*it))
+ return true;
+ }
+ return false;
}
void PluginDataRemoverHelper::Observe(
« no previous file with comments | « no previous file | content/browser/plugin_data_remover_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698