Add browser version to device status reports.

chrome/browser/policy/device_status_collector.cc

Index: chrome/browser/policy/device_status_collector.cc
diff --git a/chrome/browser/policy/device_status_collector.cc b/chrome/browser/policy/device_status_collector.cc
index 7b23d987c32cf07368300c92d497a643916f3fb1..1715838f55fe0d0e829313e06014a191dd53a3cd 100644
--- a/chrome/browser/policy/device_status_collector.cc
+++ b/chrome/browser/policy/device_status_collector.cc
@@ -10,6 +10,7 @@
 #include "chrome/browser/policy/proto/device_management_backend.pb.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
+#include "chrome/common/chrome_version_info.h"
 
 using base::Time;
 using base::TimeDelta;
@@ -154,6 +155,8 @@ void DeviceStatusCollector::GetStatus(em::DeviceStatusReportRequest* request) {
   ListPrefUpdate update(local_state_, kPrefDeviceActivePeriods);
   update.Get()->Clear();
 
+  chrome::VersionInfo version_info;
+  request->set_browser_version(version_info.CreateVersionString());

[Mattias Nissler (ping if slow), 2012/01/18 13:54:01]

That's cool as long as the string is not getting parsed into its components on
the server side. If they plan on doing that, we should rather send the
individual components of the full version string.

[Patrick Dubroy, 2012/01/18 14:58:09]

Good point -- I think we actually only want the version number.

On 2012/01/18 13:54:01, Mattias Nissler wrote:

   request->set_os_version(os_version_);
   request->set_firmware_version(firmware_version_);
 }