Don't add fake flash when command-line flash exists.

chrome/common/chrome_content_client.cc

Index: chrome/common/chrome_content_client.cc
diff --git a/chrome/common/chrome_content_client.cc b/chrome/common/chrome_content_client.cc
index d13f2ede82618aa1fe64516224d7ee5196772d1d..616c9b15d2dfe692618f4e9fd7b24e3dc4e75d74 100644
--- a/chrome/common/chrome_content_client.cc
+++ b/chrome/common/chrome_content_client.cc
@@ -280,13 +280,12 @@ content::PepperPluginInfo CreatePepperFlashInfo(const base::FilePath& path,
   return plugin;
 }
 
-void AddPepperFlashFromCommandLine(
-    std::vector<content::PepperPluginInfo>* plugins) {
+bool GetCommandLinePepperFlash(content::PepperPluginInfo* plugin) {
   const base::CommandLine::StringType flash_path =
       base::CommandLine::ForCurrentProcess()->GetSwitchValueNative(
           switches::kPpapiFlashPath);
   if (flash_path.empty())
-    return;
+    return false;
 
   // Also get the version from the command-line. Should be something like 11.2
   // or 11.2.123.45.
@@ -294,9 +293,9 @@ void AddPepperFlashFromCommandLine(
       base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
           switches::kPpapiFlashVersion);
 
-  plugins->push_back(
-      CreatePepperFlashInfo(base::FilePath(flash_path),
-                            flash_version, false, true, false));
+  *plugin = CreatePepperFlashInfo(base::FilePath(flash_path),
+                                  flash_version, false, true, false);
+  return true;
 }
 
 #if defined(OS_LINUX)
@@ -480,10 +479,9 @@ void ChromeContentClient::AddPepperPlugins(
     std::vector<content::PepperPluginInfo>* plugins) {
 #if defined(ENABLE_PLUGINS)
   ComputeBuiltInPlugins(plugins);
-  AddPepperFlashFromCommandLine(plugins);
 
 #if defined(OS_LINUX)
-  // Depending on the sandbox configurtion, the user data directory
+  // Depending on the sandbox configuration, the user data directory
   // is not always available. If it is not available, do not try and load any
   // flash plugin. The flash player, if any, preloaded before the sandbox
   // initialization will continue to be used.
@@ -493,6 +491,10 @@ void ChromeContentClient::AddPepperPlugins(
 #endif  // defined(OS_LINUX)
 
   ScopedVector<content::PepperPluginInfo> flash_versions;
+  std::unique_ptr<content::PepperPluginInfo> command_line_flash(

[Nico, 2016/09/27 21:09:14]

why is this on the heap?

Also, why the api changin in GetCommandLinePepperFlash()? Couldn't you just pass
flash_versions instead of plugins to the old GetCommandLinePepperFlash()?

[Greg K, 2016/09/27 21:12:29]

FWIW, since the other functions take a PepperPluginInfo*, I like the continuity
here, unless we have them all take the flash_versions list.

[Nico, 2016/09/27 21:13:40]

I like CLs that do one thing per CL: Either change behavior, or refactor code
and don't change behavior. Since this CL says it's a "change behavior" CL I'd
prefer if it was limited to doing the behavior change it's advertised to do in
the simplest way possible :-)

[waffles, 2016/09/27 21:44:52]

Passing flash_versions was my first thought, but since it's a ScopedVector
instead of a std::vector, I still needed to change the signature of that
function; while I was there it started to bother me that the signature of the
function differed from GetComponentUpdatedPepperFlash and GetSystemPepperFlash,
thus the alteration.

I think it's a small refactoring, but I can undo it or split it into a separate
CL if you insist.

Re: heap, it feels strange to me to use ScopedVector with objects not in the
heap, although I agree that the entire ScopedVector here could be rethought.
That seemed like a much larger refactor, so I landed on this middle ground.

[Nico, 2016/09/27 21:46:24]

Ah, I had missed that flash_versions is a ScopedVector, my bad, sorry :-/ lgtm
then.

+      new content::PepperPluginInfo);
+  if (GetCommandLinePepperFlash(command_line_flash.get()))
+    flash_versions.push_back(command_line_flash.release());
 
 #if defined(OS_LINUX)
   std::unique_ptr<content::PepperPluginInfo> component_flash(