Move the pepper flash to the top of the flash plugin list, if the client is within the enable-by-de…

chrome/browser/component_updater/pepper_flash_component_installer.cc

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..8e36d7bed094b7c5f3fb57fdf127a56e1c68dbe0 100644
--- a/chrome/browser/component_updater/pepper_flash_component_installer.cc
+++ b/chrome/browser/component_updater/pepper_flash_component_installer.cc
@@ -178,12 +178,12 @@ 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);
+
   PluginService::GetInstance()->RegisterInternalPlugin(
-      plugin_info.ToWebPluginInfo(), false);
+      plugin_info.ToWebPluginInfo(), enable_by_default);

[Bernhard Bauer, 2012/03/05 22:23:09]

Could you move this into a separate variable |add_to_front| or something so that
the meaning of the parameter is clear?

[yzshen1, 2012/03/05 22:53:37]

Done.

   PluginService::GetInstance()->RefreshPlugins();
 }