Include the language (locale) info in the update ping

chrome/browser/extensions/webstore_installer_unittest.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 <string>
 
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/extensions/webstore_installer.h"
-#include "chrome/common/omaha_query_params/omaha_query_params.h"
+#include "chrome/browser/omaha_query_params/omaha_query_params.h"
 #include "extensions/common/id_util.h"
 #include "net/base/escape.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::StringPrintf;
 using chrome::OmahaQueryParams;
 
 namespace extensions {
 
 // Returns true if |target| is found in |source|.
 bool Contains(const std::string& source, const std::string& target) {
   return source.find(target) != std::string::npos;
 }
 
 TEST(WebstoreInstallerTest, PlatformParams) {

[Sorin Jianu, 2014/02/13 19:06:55]

Should this test be updated to include the lang query string parameter?

[Devlin, 2014/02/14 00:26:42]

Done.

   std::string id = extensions::id_util::GenerateId("some random string");
   std::string source = "inline";
   GURL url = WebstoreInstaller::GetWebstoreInstallURL(id,
       WebstoreInstaller::INSTALL_SOURCE_INLINE);
   std::string query = url.query();
   EXPECT_TRUE(Contains(query,StringPrintf("os=%s", OmahaQueryParams::getOS())));
   EXPECT_TRUE(Contains(query,StringPrintf("arch=%s",
                                           OmahaQueryParams::getArch())));
   EXPECT_TRUE(Contains(query,StringPrintf("nacl_arch=%s",
                                           OmahaQueryParams::getNaclArch())));
   EXPECT_TRUE(Contains(query,net::EscapeQueryParamValue(
       StringPrintf("installsource=%s", source.c_str()),
       true)));
 }
 
 }  // namespace extensions