Add SourceUI field to permission report

chrome/browser/permissions/permission_context_base.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_context_base.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/logging.h"
@@ skipping 199 matching lines @@
     bool persist,
     ContentSetting content_setting) {
 #if !defined(OS_ANDROID)
   // Infobar persistence and its related UMA is tracked on the infobar
   // controller directly.
   if (persist) {
     DCHECK(content_setting == CONTENT_SETTING_ALLOW ||
            content_setting == CONTENT_SETTING_BLOCK);
     if (content_setting == CONTENT_SETTING_ALLOW)
       PermissionUmaUtil::PermissionGranted(permission_type_, requesting_origin,
-                                           profile_);
+                                           PROMPT, profile_);

[raymes, 2016/07/12 00:54:36]

As discussed, I think in most cases we don't need to pass the SourceUI here
because we can infer it from the operation (e.g. PermissionGranted can only be
called for prompts).

[kcarattini, 2016/07/12 01:01:42]

I assume that we don't call it for anything but the prompt now, but will that
change? E.g. OIB or Site Settings?

[stefanocs, 2016/07/12 04:33:33]

Done.

     else
       PermissionUmaUtil::PermissionDenied(permission_type_, requesting_origin,
-                                          profile_);
+                                          PROMPT, profile_);
   } else {
     DCHECK_EQ(content_setting, CONTENT_SETTING_DEFAULT);
     PermissionUmaUtil::PermissionDismissed(permission_type_, requesting_origin,
-                                           profile_);
+                                           PROMPT, profile_);
   }
 #endif
 
   NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
                       persist, content_setting);
 }
 
 #if defined(OS_ANDROID)
 PermissionQueueController* PermissionContextBase::GetQueueController() {
   return permission_queue_controller_.get();
@@ skipping 49 matching lines @@
                                       content_setting);
 }
 
 bool PermissionContextBase::IsPermissionKillSwitchOn() const {
   const std::string param = variations::GetVariationParamValue(
       kPermissionsKillSwitchFieldStudy,
       PermissionUtil::GetPermissionString(permission_type_));
 
   return param == kPermissionsKillSwitchBlockedValue;
 }