Adding version info to the client config request

components/data_reduction_proxy/proto/client_config.proto

 // Copyright 2015 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.
 
 syntax = "proto2";
 
 option optimize_for = LITE_RUNTIME;
 
 package data_reduction_proxy;
 
@@ skipping 91 matching lines @@
   // 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;
+}