Record metrics for permission prompt acceptance vs. user gesture.

chrome/browser/permissions/permission_bubble_request_impl.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/permissions/permission_bubble_request_impl.h"
 
 #include "build/build_config.h"
 #include "chrome/browser/permissions/permission_context_base.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/url_formatter/elide_url.h"
 #include "grit/theme_resources.h"
 #include "net/base/escape.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/vector_icons_public.h"
 
 PermissionBubbleRequestImpl::PermissionBubbleRequestImpl(
     const GURL& request_origin,
     content::PermissionType permission_type,
+    bool has_gesture,
     const PermissionDecidedCallback& permission_decided_callback,
     const base::Closure delete_callback)
     : request_origin_(request_origin),
       permission_type_(permission_type),

[tsergeant, 2016/07/06 00:28:47]

Shouldn't has_gesture_ be initialized in here?

[benwells, 2016/07/06 01:14:01]

Yes, indeed it should! Done.

       permission_decided_callback_(permission_decided_callback),
       delete_callback_(delete_callback),
       is_finished_(false),
       action_taken_(false) {}
 
 PermissionBubbleRequestImpl::~PermissionBubbleRequestImpl() {
   DCHECK(is_finished_);
   if (!action_taken_)
     PermissionUmaUtil::PermissionIgnored(permission_type_, request_origin_);
 }
@@ skipping 113 matching lines @@
       return PermissionBubbleType::PERMISSION_PUSH_MESSAGING;
 #if defined(OS_CHROMEOS)
     case content::PermissionType::PROTECTED_MEDIA_IDENTIFIER:
       return PermissionBubbleType::PERMISSION_PROTECTED_MEDIA_IDENTIFIER;
 #endif
     default:
       NOTREACHED();
       return PermissionBubbleType::UNKNOWN;
   }
 }
+
+PermissionBubbleGestureType PermissionBubbleRequestImpl::GetGestureType()
+    const {
+  return has_gesture_ ? PermissionBubbleGestureType::GESTURE
+                      : PermissionBubbleGestureType::NO_GESTURE;
+}