Add gesture type value from desktop prompt to permission report

chrome/browser/push_messaging/push_messaging_permission_context.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/push_messaging/push_messaging_permission_context.h"
 
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/permissions/permission_manager.h"
 #include "chrome/browser/permissions/permission_request_id.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
@@ skipping 100 matching lines @@
   DCHECK_NE(notification_status, blink::mojom::PermissionStatus::ASK);
 
   ContentSetting push_content_setting =
       HostContentSettingsMapFactory::GetForProfile(profile_)
           ->GetContentSettingAndMaybeUpdateLastUsage(
               requesting_origin, embedding_origin, content_settings_type(),
               std::string());
 
   if (push_content_setting == CONTENT_SETTING_BLOCK) {
     DVLOG(1) << "Push permission was explicitly blocked.";
-    PermissionUmaUtil::PermissionDenied(permission_type(), requesting_origin,
-                                        profile_);
+    // TODO(stefanocs): Pass in the actual |user_gesture| once push messaging
+    // metrics has been fixed.
+    PermissionUmaUtil::PermissionDenied(permission_type(),

[kcarattini, 2016/07/18 07:04:33]

Can you specify what you mean by "push messaging metrics has been fixed"? Is
there a specific bug you are referring to? When do you estimate it will be
fixed?

[stefanocs, 2016/07/18 07:19:33]

Done. I think Raymes is working on this one. Raymes, is there a specific bug
tracker for this one?

[raymes, 2016/07/18 07:31:59]

Once this CL is landed it will happen automatically:
https://codereview.chromium.org/2149883002/

[stefanocs, 2016/07/18 07:40:25]

In that case, I will probably just remove the comment.

+                                        false /* user_gesture */,
+                                        requesting_origin, profile_);
     NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                         true /* persist */, CONTENT_SETTING_BLOCK);
     return;
   }
 
   if (notification_status == blink::mojom::PermissionStatus::DENIED) {
     DVLOG(1) << "Notification permission has not been granted.";
     NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                         false /* persist */, CONTENT_SETTING_BLOCK);
     return;
   }
 
-  PermissionUmaUtil::PermissionGranted(permission_type(), requesting_origin,
-                                       profile_);
+  // TODO(stefanocs): Pass in the actual |user_gesture| once push messaging
+  // metrics has been fixed.
+  PermissionUmaUtil::PermissionGranted(
+      permission_type(), false /* user_gesture */, requesting_origin, profile_);
   NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                       true /* persist */, CONTENT_SETTING_ALLOW);
 }