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 72 matching lines @@
     NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                         false /* persist */, CONTENT_SETTING_BLOCK);
     return;
   }
 
   PermissionManager::Get(profile_)->RequestPermission(
       content::PermissionType::NOTIFICATIONS, web_contents->GetMainFrame(),
       requesting_origin, user_gesture,
       base::Bind(&PushMessagingPermissionContext::DecidePushPermission,
                  weak_factory_ui_thread_.GetWeakPtr(), id, requesting_origin,
-                 embedding_origin, callback));
+                 embedding_origin, user_gesture, callback));
 #else
   NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                       false /* persist */, CONTENT_SETTING_BLOCK);
 #endif
 }
 
 bool PushMessagingPermissionContext::IsRestrictedToSecureOrigins() const {
   return true;
 }
 
 void PushMessagingPermissionContext::DecidePushPermission(
     const PermissionRequestID& id,
     const GURL& requesting_origin,
     const GURL& embedding_origin,
+    bool user_gesture,

[raymes, 2016/07/18 00:58:43]

I would say don't bother passing this for now since those metrics are wrong
anyway and I'm making some changes to the code. Perhaps just pass false and add
a TODO below to pass the correct value.

[stefanocs, 2016/07/18 04:28:45]

Done.

     const BrowserPermissionCallback& callback,
     blink::mojom::PermissionStatus notification_status) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   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_);
+    PermissionUmaUtil::PermissionDenied(permission_type(), 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_);
+  PermissionUmaUtil::PermissionGranted(permission_type(), user_gesture,
+                                       requesting_origin, profile_);
   NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                       true /* persist */, CONTENT_SETTING_ALLOW);
 }