Move OmahaQueryParams to a component and add a delegate interface.

components/omaha_query_params/omaha_query_params_unittest.cc

Index: components/omaha_query_params/omaha_query_params_unittest.cc
diff --git a/chrome/browser/omaha_query_params/omaha_query_params_unittest.cc b/components/omaha_query_params/omaha_query_params_unittest.cc
similarity index 60%
rename from chrome/browser/omaha_query_params/omaha_query_params_unittest.cc
rename to components/omaha_query_params/omaha_query_params_unittest.cc
index e8a736aea300f7aabd1d7b11b1b791037f0b6c58..41161a24336e4303545663c73307a33f701e64a4 100644
--- a/chrome/browser/omaha_query_params/omaha_query_params_unittest.cc
+++ b/components/omaha_query_params/omaha_query_params_unittest.cc
@@ -2,21 +2,25 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/omaha_query_params/omaha_query_params.h"
-
 #include "base/strings/stringprintf.h"
-#include "chrome/common/chrome_version_info.h"
+#include "components/omaha_query_params/omaha_query_params.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using base::StringPrintf;
 
-namespace chrome {
+namespace {
 
 bool Contains(const std::string& source, const std::string& target) {
   return source.find(target) != std::string::npos;
 }
 
-void TestParams(OmahaQueryParams::ProdId prod_id) {
+class TestOmahaQueryParamsDelegate : public OmahaQueryParamsDelegate {
+  virtual std::string GetExtraParams() OVERRIDE { return "&cat=dog"; }
+};
+
+}  // namespace
+
+void TestParams(OmahaQueryParams::ProdId prod_id, bool extra_params) {
   std::string params = OmahaQueryParams::Get(prod_id);
 
   // This doesn't so much test what the values are (since that would be an
@@ -30,20 +34,17 @@ void TestParams(OmahaQueryParams::ProdId prod_id) {
   EXPECT_TRUE(Contains(
       params,
       StringPrintf("prod=%s", OmahaQueryParams::GetProdIdString(prod_id))));
-  EXPECT_TRUE(Contains(
-      params,
-      StringPrintf("prodchannel=%s", OmahaQueryParams::GetChannelString())));
-  EXPECT_TRUE(Contains(
-      params,
-      StringPrintf("prodversion=%s", chrome::VersionInfo().Version().c_str())));
-  EXPECT_TRUE(Contains(
-      params,
-      StringPrintf("lang=%s", OmahaQueryParams::GetLang())));
+  if (extra_params)
+    EXPECT_TRUE(Contains(params, "cat=dog"));
 }
 
-TEST(OmahaQueryParams, GetOmahaQueryParams) {
-  TestParams(OmahaQueryParams::CRX);
-  TestParams(OmahaQueryParams::CHROME);
-}
+TEST(OmahaQueryParamsTest, GetParams) {
+  TestParams(OmahaQueryParams::CRX, false);
+  TestParams(OmahaQueryParams::CHROME, false);
+
+  TestOmahaQueryParamsDelegate delegate;
+  OmahaQueryParamsDelegate::Set(&delegate);
 
-}  // namespace chrome
+  TestParams(OmahaQueryParams::CRX, true);
+  TestParams(OmahaQueryParams::CHROME, true);
+}