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

Unified Diff: chrome/browser/component_updater/pepper_flash_component_installer.cc

Issue 9600029: Move the pepper flash to the top of the flash plugin list, if the client is within the enable-by-de… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: In response to Bernhard's comments. Created 8 years, 10 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/component_updater/pepper_flash_component_installer.cc
diff --git a/chrome/browser/component_updater/pepper_flash_component_installer.cc b/chrome/browser/component_updater/pepper_flash_component_installer.cc
index cedf5390edb2d6a0c5d0b0a8188582e68527e4af..82bd55a4a0fbdbbaa6be75eba0ca8c0685057cd7 100644
--- a/chrome/browser/component_updater/pepper_flash_component_installer.cc
+++ b/chrome/browser/component_updater/pepper_flash_component_installer.cc
@@ -178,12 +178,13 @@ void RegisterPepperFlashWithChrome(const FilePath& path,
content::PepperPluginInfo plugin_info;
if (!MakePepperFlashPluginInfo(path, version, true, &plugin_info))
return;
- if (fresh_install) {
- PluginPrefs::EnablePluginGlobally(IsPepperFlashEnabledByDefault(),
- plugin_info.path);
- }
+ bool enable_by_default = IsPepperFlashEnabledByDefault();
+ if (fresh_install)
+ PluginPrefs::EnablePluginGlobally(enable_by_default, plugin_info.path);
+
+ bool add_to_front = enable_by_default;
PluginService::GetInstance()->RegisterInternalPlugin(
- plugin_info.ToWebPluginInfo(), false);
+ plugin_info.ToWebPluginInfo(), add_to_front);
PluginService::GetInstance()->RefreshPlugins();
}
« 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