Add user gesture field to permission report

chrome/browser/permissions/permission_uma_util.cc

Index: chrome/browser/permissions/permission_uma_util.cc
diff --git a/chrome/browser/permissions/permission_uma_util.cc b/chrome/browser/permissions/permission_uma_util.cc
index 7eae053925997d1ba44594934e964669537d0829..0206a20695e3b9d491a9a99ad443ee881136847c 100644
--- a/chrome/browser/permissions/permission_uma_util.cc
+++ b/chrome/browser/permissions/permission_uma_util.cc
@@ -339,10 +339,11 @@ void PermissionUmaUtil::RecordPermissionAction(PermissionType permission,
   if (IsOptedIntoPermissionActionReporting(profile)) {
     // TODO(stefanocs): Add browsertests to make sure the reports are being
     // sent.
+    // TODO(stefanocs): Get the actual |user_gesture| from permission layer.
     g_browser_process->safe_browsing_service()
         ->ui_manager()
         ->ReportPermissionAction(requesting_origin, permission, action,
-                                 source_ui);
+                                 source_ui, false);

[raymes, 2016/07/18 00:31:25]

Rather than using a bool, it's probably better to define an enum. This serves as
documentation of what the argument is for (rather than just true/false).

[raymes, 2016/07/18 00:35:11]

I see most of the other code just uses a bool for this, so I'm ok sticking with
a bool. But we need to include inline comments denoting what the argument is.
e.g. in this case, it would be:

, false /* user_gesture */);

[stefanocs, 2016/07/18 00:48:07]

Done. I was planning on doing this before but I wasn't clear whether to use
vector<RequestTrigger> or just RequestTrigger and since most of the code in
permission layer uses bool, I wasn't sure where we going to convert it into a
request trigger. So I decided to stick with this first.

   }
 
   bool secure_origin = content::IsOriginSecure(requesting_origin);