Content Settings: Add RevocationObserver to measure when permissions are revoked

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"
 #include "content/public/browser/permission_type.h"
 
+

[Bernhard Bauer, 2016/03/22 10:13:29]

Nit: This empty line is unnecessary.

[tsergeant, 2016/03/22 23:55:20]

Done.

 using content::PermissionType;
 
 // The returned strings must match the RAPPOR metrics in rappor.xml,
 // and any Field Trial configs for the Permissions kill switch e.g.
 // Permissions.Action.Geolocation etc..
 std::string PermissionUtil::GetPermissionString(
     content::PermissionType permission) {
   switch (permission) {
     case content::PermissionType::GEOLOCATION:
       return "Geolocation";
@@ skipping 42 matching lines @@
     *out = PermissionType::BACKGROUND_SYNC;
 #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
   } else if (type == CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER) {
     *out = PermissionType::PROTECTED_MEDIA_IDENTIFIER;
 #endif
   } else {
     return false;
   }
   return true;
 }
+
+bool PermissionUtil::SetContentSettingRecordRevocation(
+    Profile* profile,
+    const GURL& primary_url,
+    const GURL& secondary_url,
+    ContentSettingsType content_type,
+    std::string resource_identifier,
+    ContentSetting setting) {
+  HostContentSettingsMap* map =
+      HostContentSettingsMapFactory::GetForProfile(profile);
+  ContentSetting previous_value = map->GetContentSetting(
+      primary_url, secondary_url, content_type, resource_identifier);
+
+  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)) {
+      PermissionUmaUtil::PermissionRevoked(permission_type, primary_url);
+      return true;
+    }
+  }
+  return false;
+}