Add revocation metrics from OIB and content setting

chrome/browser/ui/website_settings/website_settings.cc

Index: chrome/browser/ui/website_settings/website_settings.cc
diff --git a/chrome/browser/ui/website_settings/website_settings.cc b/chrome/browser/ui/website_settings/website_settings.cc
index 5853d3e4094cda5b3bf438ef21ed1433af29b61d..a5c77c35ed24241a5222980fb71b634c8b700794 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -288,15 +288,6 @@ void WebsiteSettings::OnSitePermissionChanged(ContentSettingsType type,
     UMA_HISTOGRAM_ENUMERATION(
         "WebsiteSettings.OriginInfo.PermissionChanged.Blocked", histogram_value,
         num_values);
-    // Trigger Rappor sampling if it is a permission revoke action.
-    // TODO(tsergeant): Integrate this with the revocation recording performed

[kcarattini, 2016/07/26 03:54:20]

Tim, is this TODO completed now? I'm not really sure what you meant by it.

[tsergeant, 2016/07/26 04:09:30]

Yup, it's also done. It's referring to using the same PermissionUtil revocation
helper here as everywhere else.

-    // in the permissions layer. See crbug.com/469221.
-    content::PermissionType permission_type;
-    if (PermissionUtil::GetPermissionType(type, &permission_type)) {
-      PermissionUmaUtil::PermissionRevoked(permission_type,
-                                           PermissionSourceUI::OIB,
-                                           this->site_url_, this->profile_);
-    }
   }
 
   // This is technically redundant given the histogram above, but putting the
@@ -304,6 +295,10 @@ void WebsiteSettings::OnSitePermissionChanged(ContentSettingsType type,
   // compare it against other kinds of actions in WebsiteSettings[PopupView].
   RecordWebsiteSettingsAction(WEBSITE_SETTINGS_CHANGED_PERMISSION);
 
+  PermissionUtil::ScopedRevocationReporter scoped_revocation_reporter(
+      this->profile_, this->site_url_, this->site_url_, type,
+      PermissionSourceUI::OIB);
+
   content_settings_->SetNarrowestContentSetting(site_url_, site_url_, type,
                                                 setting);