Report built_in_version to Omaha for Flash updates.

chrome/browser/component_updater/pepper_flash_component_installer.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/component_updater/pepper_flash_component_installer.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
@@ skipping 323 matching lines @@
 void FlashComponentInstallerTraits::GetHash(std::vector<uint8_t>* hash) const {
   hash->assign(kSha2Hash, kSha2Hash + arraysize(kSha2Hash));
 }
 
 std::string FlashComponentInstallerTraits::GetName() const {
   return "Adobe Flash Player";
 }
 
 update_client::InstallerAttributes
 FlashComponentInstallerTraits::GetInstallerAttributes() const {
-  return update_client::InstallerAttributes();
+  // For Chrome OS, send the built-in flash player version to the server,
+  // otherwise it will serve component updates of outdated flash players.
+  update_client::InstallerAttributes attrs =
+      update_client::InstallerAttributes();
+#if defined(OS_CHROMEOS)
+  std::string flash_version =
+      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+          switches::kPpapiFlashVersion);
+  bool did_insert =
+      attrs.insert(std::make_pair("built_in_version", flash_version)).second;

[waffles, 2017/02/24 18:35:51]

Will this generated an unused variable warning in release builds?

[Greg K, 2017/02/24 19:03:59]

That's a fair point, let's do the DCHECK() directly.

+  DCHECK(did_insert);
+#endif  // #defined(OS_CHROMEOS)
+  return attrs;
 }
 
 std::vector<std::string> FlashComponentInstallerTraits::GetMimeTypes() const {
   std::vector<std::string> mime_types;
   mime_types.push_back("application/x-shockwave-flash");
   mime_types.push_back("application/futuresplash");
   return mime_types;
 }
 #endif  // defined(GOOGLE_CHROME_BUILD)
 
@@ skipping 15 matching lines @@
   std::unique_ptr<ComponentInstallerTraits> traits(
       new FlashComponentInstallerTraits);
   // |cus| will take ownership of |installer| during installer->Register(cus).
   DefaultComponentInstaller* installer =
       new DefaultComponentInstaller(std::move(traits));
   installer->Register(cus, base::Closure());
 #endif  // defined(GOOGLE_CHROME_BUILD)
 }
 
 }  // namespace component_updater