Correcting the version field in the Chrome-Proxy header to be the chromium build and patch number.

chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.cc

 // Copyright 2014 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 "chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.h"
 
+#include "base/strings/string_split.h"
+#include "base/strings/string_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_configurator.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "components/data_reduction_proxy/browser/data_reduction_proxy_configurator.h"
 #include "components/data_reduction_proxy/browser/data_reduction_proxy_params.h"
 #include "components/data_reduction_proxy/browser/data_reduction_proxy_settings.h"
 
 using data_reduction_proxy::DataReductionProxyParams;
 using data_reduction_proxy::DataReductionProxySettings;
@@ skipping 14 matching lines @@
       configurator(new DataReductionProxyChromeConfigurator(prefs));
   SetProxyConfigurator(configurator.Pass());
   DataReductionProxySettings::InitDataReductionProxySettings(
       prefs,
       g_browser_process->local_state(),
       ProfileManager::GetActiveUserProfile()->GetRequestContext());
 
   SetDataReductionProxyAlternativeEnabled(
       DataReductionProxyParams::IsIncludedInAlternativeFieldTrial());
 }
+
+// static
+std::string DataReductionProxyChromeSettings::GetBuildAndPatchNumber() {
+  chrome::VersionInfo version_info;
+  std::vector<std::string> version_parts;
+  base::SplitString(version_info.Version(), '.', &version_parts);
+  if (version_parts.size() != 4) {
+    return "";
+  }

[Lei Zhang, 2014/08/01 19:11:57]

nit: you don't need the braces.

[megjablon, 2014/08/01 20:35:26]

Done.

+  version_parts.erase(version_parts.begin(), version_parts.begin() + 2);

[Lei Zhang, 2014/08/01 19:11:57]

nit: isn't the rest of the function just: return version_parts[2] +
version_parts[3] ?

[megjablon, 2014/08/01 20:35:26]

Ya I had this originally and like it better. Switching back.

+  return JoinString(version_parts, '.');
+}
+
+// static
+std::string DataReductionProxyChromeSettings::GetClient() {
+#if defined(OS_ANDROID)
+  return data_reduction_proxy::kClientChromeAndroid;
+#elif defined(OS_IOS)
+  return data_reduction_proxy::kClientChromeIOS;
+#else
+  return "";
+#endif
+}