Adding version info to the client config request

components/data_reduction_proxy/proto/client_config.proto

Index: components/data_reduction_proxy/proto/client_config.proto
diff --git a/components/data_reduction_proxy/proto/client_config.proto b/components/data_reduction_proxy/proto/client_config.proto
index 98cc093e0d4b565d1ce9fd9d9a305ab70c243761..43256647170102012f6fc3e1c3c169b0ece1be4e 100644
--- a/components/data_reduction_proxy/proto/client_config.proto
+++ b/components/data_reduction_proxy/proto/client_config.proto
@@ -102,11 +102,31 @@ message ProxyServer {
   // The host name for the proxy server.
   optional string host = 2;
   // The port number for the proxy server.
   optional int32 port = 3;
 }
 
 // Request object to create a client configuration object.
 message CreateClientConfigRequest {
   // A previous per-session key that was assigned by the service.
   optional string session_key = 1;
+
+  // Build version information.
+  optional VersionInfo version_info = 2;
 }
+
+// Build version information.
+message VersionInfo {
+  // The client's platform type. See
+  // components/data_reduction_proxy/core/common/data_reduction_proxy_util.h for
+  // allowed strings in CLIENT_ENUMS_LIST.
+  optional string client = 1;
+
+  // The version number, e.g. 50

[tbansal1, 2016/06/29 23:56:38]

This comment is slightly confusing. From the value "50", it seems you are
returning milestone, but you are returning the build number, which is generally
a 4-digit number. 
s/version/build/
s/50/2171/

[RyanSturm, 2016/06/30 00:02:22]

Done.

+  optional int32 build = 2;
+
+  // The patch number of the chromium client: always a non-negative integer.
+  optional int32 patch = 3;
+
+  // The production channel, e.g. Canary, Dev, Beta, Stable.

[tbansal1, 2016/06/29 23:56:38]

s/Canary/canary/
s/Dev/dev/
....
assuming I am reading
https://cs.chromium.org/chromium/src/components/version_info/version_info.cc?...
correctly.

[RyanSturm, 2016/06/30 00:02:22]

Done.

+  optional string channel = 4;
+}