Add revocation metrics from OIB and content setting

chrome/browser/permissions/permission_util_unittest.cc

 // Copyright 2016 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 "base/test/histogram_tester.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/browser/permissions/permission_util.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 class PermissionUtilTest : public testing::Test {
   content::TestBrowserThreadBundle thread_bundle_;
 };
 
 TEST_F(PermissionUtilTest, ScopedRevocationReporter) {
   TestingProfile profile;
   // TODO(tsergeant): Add more comprehensive tests of PermissionUmaUtil.
   base::HistogramTester histograms;
   HostContentSettingsMap* map =
       HostContentSettingsMapFactory::GetForProfile(&profile);
   GURL host("https://example.com");
+  ContentSettingsPattern host_pattern =
+      ContentSettingsPattern::FromURLNoWildcard(host);
+  ContentSettingsPattern host_containing_wildcards_pattern =
+      ContentSettingsPattern::FromString("https://[*.]example.com/");
   ContentSettingsType type = CONTENT_SETTINGS_TYPE_GEOLOCATION;
   PermissionSourceUI source_ui = PermissionSourceUI::SITE_SETTINGS;
 
   // Allow->Block triggers a revocation.
   map->SetContentSettingDefaultScope(host, host, type, std::string(),
                                      CONTENT_SETTING_ALLOW);
   {
     PermissionUtil::ScopedRevocationReporter scoped_revocation_reporter(
         &profile, host, host, type, source_ui);
     map->SetContentSettingDefaultScope(host, host, type, std::string(),
@@ skipping 26 matching lines @@
   // Allow->Default does not trigger a revocation when default is 'allow'.
   map->SetDefaultContentSetting(type, CONTENT_SETTING_ALLOW);
   {
     PermissionUtil::ScopedRevocationReporter scoped_revocation_reporter(
         &profile, host, host, type, source_ui);
     map->SetContentSettingDefaultScope(host, host, type, std::string(),
                                        CONTENT_SETTING_DEFAULT);
   }
   histograms.ExpectBucketCount("Permissions.Action.Geolocation",
                                PermissionAction::REVOKED, 2);
+
+  // Allow->Block with url pattern string triggers a revocation.
+  map->SetContentSettingDefaultScope(host, host, type, std::string(),
+                                     CONTENT_SETTING_ALLOW);
+  {
+    PermissionUtil::ScopedRevocationReporter scoped_revocation_reporter(
+        &profile, host_pattern, host_pattern, type, source_ui);
+    map->SetContentSettingCustomScope(host_pattern, host_pattern, type,
+                                      std::string(), CONTENT_SETTING_BLOCK);
+  }
+  histograms.ExpectBucketCount("Permissions.Action.Geolocation",
+                               PermissionAction::REVOKED, 3);
+
+  // Allow->Block with non url pattern string does not trigger a revocation.
+  map->SetContentSettingDefaultScope(host, host, type, std::string(),
+                                     CONTENT_SETTING_ALLOW);
+  {
+    PermissionUtil::ScopedRevocationReporter scoped_revocation_reporter(
+        &profile, host_containing_wildcards_pattern, host_pattern, type,
+        source_ui);
+    map->SetContentSettingCustomScope(host_containing_wildcards_pattern,
+                                      host_pattern, type, std::string(),
+                                      CONTENT_SETTING_BLOCK);
+  }
+  histograms.ExpectBucketCount("Permissions.Action.Geolocation",
+                               PermissionAction::REVOKED, 3);
 }