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

Unified Diff: chrome/browser/plugins/plugin_prefs.cc

Issue 1633363002: Remove NPAPI Flash migration code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: static_cast Created 4 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/plugins/plugin_prefs.cc
diff --git a/chrome/browser/plugins/plugin_prefs.cc b/chrome/browser/plugins/plugin_prefs.cc
index 11b74a071ad45a9e5998aff956f63133db0a8b9a..f327189060209589fcf5ddc79f8ffb7837b2db6c 100644
--- a/chrome/browser/plugins/plugin_prefs.cc
+++ b/chrome/browser/plugins/plugin_prefs.cc
@@ -59,24 +59,6 @@ bool IsComponentUpdatedPepperFlash(const base::FilePath& plugin) {
return false;
}
-// Returns true if |path| looks like the path to an NPAPI Flash plugin.
-bool IsNpapiFlashPath(const base::FilePath& path) {
- base::FilePath npapi_flash;
- // Check NPAPI Flash is installed.
- if (!PathService::Get(chrome::FILE_FLASH_SYSTEM_PLUGIN, &npapi_flash))
- return false;
- // Check for same architecture NPAPI Flash.
- if (base::FilePath::CompareEqualIgnoreCase(path.value(), npapi_flash.value()))
- return true;
-#if defined(OS_WIN)
- // Fuzzy check for NPAPI Flash on Windows.
- base::FilePath::StringType kSwfPrefix = FILE_PATH_LITERAL("NPSWF");
- if (path.BaseName().value().compare(0, kSwfPrefix.size(), kSwfPrefix) == 0)
- return true;
-#endif
- return false;
-}
-
} // namespace
PluginPrefs::PluginState::PluginState() {
@@ -346,42 +328,18 @@ void PluginPrefs::SetPrefs(PrefService* prefs) {
prefs::kPluginsLastInternalDirectory, cur_internal_dir);
}
- bool migrate_to_pepper_flash = false;
-#if defined(OS_WIN) || defined(OS_MACOSX)
- // If NPAPI Flash is enabled while Pepper Flash is disabled, we would like to
- // turn Pepper Flash on. And we want to do it once, when NPAPI is disabled in
- // Chrome 45.
- // TODO(wfh): Remove this code once it has been run by most users, around
- // Chrome 49 or Chrome 50. See crbug.com/514250.
- if (!prefs_->GetBoolean(prefs::kNpapiFlashMigratedToPepperFlash)) {
- prefs_->SetBoolean(prefs::kNpapiFlashMigratedToPepperFlash, true);
- migrate_to_pepper_flash = true;
- }
-#endif
-
{ // Scoped update of prefs::kPluginsPluginsList.
ListPrefUpdate update(prefs_, prefs::kPluginsPluginsList);
base::ListValue* saved_plugins_list = update.Get();
if (saved_plugins_list && !saved_plugins_list->empty()) {
- // The following four variables are only valid when
- // |migrate_to_pepper_flash| is set to true.
- base::FilePath pepper_flash;
- base::DictionaryValue* pepper_flash_node = NULL;
- bool npapi_flash_enabled = false;
- if (migrate_to_pepper_flash) {
- PathService::Get(chrome::FILE_PEPPER_FLASH_PLUGIN, &pepper_flash);
- }
-
- for (base::ListValue::iterator it = saved_plugins_list->begin();
- it != saved_plugins_list->end();
- ++it) {
- if (!(*it)->IsType(base::Value::TYPE_DICTIONARY)) {
+ for (base::Value* plugin_value : *saved_plugins_list) {
+ if (!plugin_value->IsType(base::Value::TYPE_DICTIONARY)) {
LOG(WARNING) << "Invalid entry in " << prefs::kPluginsPluginsList;
continue; // Oops, don't know what to do with this item.
}
base::DictionaryValue* plugin =
- static_cast<base::DictionaryValue*>(*it);
+ static_cast<base::DictionaryValue*>(plugin_value);
base::string16 group_name;
bool enabled;
if (!plugin->GetBoolean("enabled", &enabled))
@@ -433,27 +391,12 @@ void PluginPrefs::SetPrefs(PrefService* prefs) {
}
}
- if (migrate_to_pepper_flash && IsNpapiFlashPath(plugin_path)) {
- npapi_flash_enabled = enabled;
- } else if (migrate_to_pepper_flash &&
- base::FilePath::CompareEqualIgnoreCase(
- path, pepper_flash.value())) {
- if (!enabled)
- pepper_flash_node = plugin;
- }
-
plugin_state_.Set(plugin_path, enabled);
} else if (!enabled && plugin->GetString("name", &group_name)) {
// Otherwise this is a list of groups.
plugin_group_state_[group_name] = false;
}
}
-
- if (npapi_flash_enabled && pepper_flash_node) {
- DCHECK(migrate_to_pepper_flash);
- pepper_flash_node->SetBoolean("enabled", true);
- plugin_state_.Set(pepper_flash, true);
- }
} else {
// If the saved plugin list is empty, then the call to UpdatePreferences()
// below failed in an earlier run, possibly because the user closed the
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698