Add field trials and platform to permission report

chrome/browser/safe_browsing/permission_reporter.cc

Index: chrome/browser/safe_browsing/permission_reporter.cc
diff --git a/chrome/browser/safe_browsing/permission_reporter.cc b/chrome/browser/safe_browsing/permission_reporter.cc
index d0364bcda25efd32910ce1bd1c9ab355edb9ced8..3470a78979fc4610c2f2d1bd53bf8ce99e44a09d 100644
--- a/chrome/browser/safe_browsing/permission_reporter.cc
+++ b/chrome/browser/safe_browsing/permission_reporter.cc
@@ -4,6 +4,7 @@
 
 #include "chrome/browser/safe_browsing/permission_reporter.h"
 #include "chrome/common/safe_browsing/permission_report.pb.h"
+#include "components/variations/active_field_trials.h"
 #include "content/public/browser/permission_type.h"
 #include "net/url_request/report_sender.h"
 
@@ -103,7 +104,23 @@ bool PermissionReporter::BuildReport(const GURL& origin,
   report.set_origin(origin.spec());
   report.set_permission(PermissionTypeToPermissionReportPermission(permission));
   report.set_action(PermissionActionToPermissionReportAction(action));
-  // TODO(stefanocs): Collect other data for the report from global variables.
+// Collect platform data.
+#if defined(OS_ANDROID)
+  report.set_platform_type(PermissionReport::ANDROID_PLATFORM);
+#elif defined(OS_MACOSX) || defined(OS_WIN) || defined(OS_CHROMEOS) || \
+    defined(OS_LINUX)
+  report.set_platform_type(PermissionReport::DESKTOP_PLATFORM);
+#else
+  report.set_platform_type(PermissionReport::PLATFORM_TYPE_UNSPECIFIED);

[raymes, 2016/06/20 01:32:11]

Perhaps add a #error here instead of setting UNSPECIFIED? I don't think it
should be compiled on other platforms but could be wrong (in which case it would
be good to know what platforms it gets compiled on).

[stefanocs, 2016/06/21 07:01:01]

Done.

+#endif
+  // Collect field trials data.
+  std::vector<variations::ActiveGroupId> active_groups_ids;
+  variations::GetFieldTrialActiveGroupIds(&active_groups_ids);

[raymes, 2016/06/20 01:32:11]

I think Kendra mentioned we may only want to add field trials that we're
interested in. But adding them all doesn't seem too bad either (and seems safer)
as long as it doesn't make the reports too big.

Kendra, wdyt?

[kcarattini, 2016/06/20 20:58:17]

Hmm, I don't want to bloat the reports with unnecessary information, but I don't
know a way to only add ones we care about (without waiting for the Chrome
release process to change the code). It seems it would be quicker just to add
them all?

+  for (auto active_group_id : active_groups_ids) {
+    PermissionReport::FieldTrial* field_trial = report.add_field_trials();
+    field_trial->set_name_id(active_group_id.name);
+    field_trial->set_group_id(active_group_id.group);
+  }
   return report.SerializeToString(output);
 }