Add SourceUI field to permission report

chrome/browser/permissions/permission_util.cc

 // Copyright 2015 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_util.h"
 
 #include "base/logging.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
@@ skipping 82 matching lines @@
   map->SetContentSettingDefaultScope(primary_url, secondary_url, content_type,
                                      resource_identifier, setting);
 
   ContentSetting final_value = map->GetContentSetting(
       primary_url, secondary_url, content_type, resource_identifier);
 
   if (previous_value == CONTENT_SETTING_ALLOW &&
       final_value != CONTENT_SETTING_ALLOW) {
     PermissionType permission_type;
     if (PermissionUtil::GetPermissionType(content_type, &permission_type)) {
+      // TODO(stefanocs): Report revocations from page action as PAGE_ACTION
+      // source UI instead of SITE_SETTINGS source UI.

[kcarattini, 2016/07/07 01:57:51]

Is this complex enough to warrant a separate cl? Let's chat offline.

[stefanocs, 2016/07/07 03:50:55]

After chatting offline, this will be fixed in the 2075053002.

       PermissionUmaUtil::PermissionRevoked(permission_type, primary_url,
-                                           profile);
+                                           SITE_SETTINGS, profile);
     }
   }
 }