Componentize ComponentUpdaterConfigurator

components/update_client/update_query_params.cc

Index: components/update_client/update_query_params.cc
diff --git a/components/update_client/update_query_params.cc b/components/update_client/update_query_params.cc
index 33bd00e197cf711cede869036974a054c40955fd..7effc237df7bd6d20472754d0ded4ab7dd9f4df9 100644
--- a/components/update_client/update_query_params.cc
+++ b/components/update_client/update_query_params.cc
@@ -15,6 +15,10 @@ namespace update_client {
 namespace {
 
 const char kUnknown[] = "unknown";
+const char kStable[] = "stable";
+const char kBeta[] = "beta";
+const char kDev[] = "dev";
+const char kCanary[] = "canary";
 
 // The request extra information is the OS and architecture, this helps
 // the server select the right package to be delivered.
@@ -128,4 +132,26 @@ void UpdateQueryParams::SetDelegate(UpdateQueryParamsDelegate* delegate) {
   g_delegate = delegate;
 }
 
+// static
+const char* UpdateQueryParams::GetChannelString(version_info::Channel channel) {

[sdefresne, 2015/08/13 12:20:00]

This is duplicating version_info::GetChannelString(). Please use the function
from the version_info component instead. Or document why it is required not to
use version_info::GetChannelString().

[droger, 2015/08/13 13:17:46]

Done, using version_info instead, which actually simplifies the code a lot.
sorin: was there a good reason to have duplication here?

[Sorin Jianu, 2015/08/13 18:24:11]

It appears that sdefresne@ componentized version_info recently (
https://codereview.chromium.org/1257633002) and for some reason, the duplicated
code in chrome_update_query_params_delegate.cc was not removed. At any rate,
this is excellent refactoring, we could eliminate the duplicated code.

Somehow related, the whole thing about query parameters and query parameters
delegate could be redone, but that is outside the scope of this change, and a
task for the owners of the respective modules.

+  switch (channel) {
+    case version_info::Channel::STABLE:
+      return kStable;
+      break;
+    case version_info::Channel::BETA:
+      return kBeta;
+      break;
+    case version_info::Channel::DEV:
+      return kDev;
+      break;
+    case version_info::Channel::CANARY:
+      return kCanary;
+      break;
+    case version_info::Channel::UNKNOWN:
+      return kUnknown;
+      break;
+  }
+  return kUnknown;
+}
+
 }  // namespace update_client