Include the language (locale) info in the update ping

chrome/browser/extensions/updater/manifest_fetch_data.cc

Index: chrome/browser/extensions/updater/manifest_fetch_data.cc
diff --git a/chrome/browser/extensions/updater/manifest_fetch_data.cc b/chrome/browser/extensions/updater/manifest_fetch_data.cc
index 1bf4250ac05063a28fdf1a0c1f7b8239b07c6082..d8d866e9949055e3919d9e26ee63e6023af08e5c 100644
--- a/chrome/browser/extensions/updater/manifest_fetch_data.cc
+++ b/chrome/browser/extensions/updater/manifest_fetch_data.cc
@@ -10,6 +10,7 @@
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/google/google_util.h"
 #include "chrome/browser/metrics/metrics_service.h"
 #include "chrome/common/omaha_query_params/omaha_query_params.h"
@@ -77,6 +78,7 @@ bool ManifestFetchData::AddExtension(std::string id, std::string version,
   parts.push_back("v=" + version);
   if (!install_source.empty())
     parts.push_back("installsource=" + install_source);
+  parts.push_back("lang=" + g_browser_process->GetApplicationLocale());

[Devlin, 2014/02/04 00:18:39]

This is also done here:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ext...

Would it make sense to put this into OmahaQueryParams at all?

[Marijn Kruisselbrink, 2014/02/10 21:06:46]

I'm not entirely sure what you're suggesting putting in OmahaQueryParams? But
yeah, it definitely looks like it might make sense to share something between
this place and WebstoreInstaller.

[Devlin, 2014/02/10 23:08:15]

The OmahaQueryParams class is where we're getting most of the parameters we're
using, like architecture, os, etc.  It seems like "lang" might be a good fit
there, since it's now used in both places.

+cpu for his thoughts.

   parts.push_back("uc");
 
   if (!update_url_data.empty()) {