Implement permissions embargo for prompts which are repeatedly ignored.

chrome/browser/permissions/permission_uma_util.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_uma_util.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 284 matching lines @@
       PermissionDecisionAutoBlocker::GetForProfile(profile);
   RecordPermissionAction(permission, PermissionAction::IGNORED,
                          PermissionSourceUI::PROMPT, gesture_type,
                          requesting_origin, profile);
   RecordPermissionPromptPriorCount(
       permission, kPermissionsPromptIgnoredPriorDismissCountPrefix,
       autoblocker->GetDismissCount(requesting_origin, permission));
   RecordPermissionPromptPriorCount(
       permission, kPermissionsPromptIgnoredPriorIgnoreCountPrefix,
       autoblocker->GetIgnoreCount(requesting_origin, permission));
-
-  // RecordPermission* methods need to be called before RecordIgnore in the
-  // blocker because they record the number of prior ignore and dismiss values,
-  // and we don't want to include the current ignore.
-  autoblocker->RecordIgnore(requesting_origin, permission);
 }
 
 void PermissionUmaUtil::PermissionRevoked(ContentSettingsType permission,
                                           PermissionSourceUI source_ui,
                                           const GURL& revoked_origin,
                                           Profile* profile) {
   // TODO(tsergeant): Expand metrics definitions for revocation to include all
   // permissions.
   if (permission == CONTENT_SETTINGS_TYPE_NOTIFICATIONS ||
       permission == CONTENT_SETTINGS_TYPE_GEOLOCATION ||
@@ skipping 16 matching lines @@
 
 void PermissionUmaUtil::RecordEmbargoPromptSuppressionFromSource(
     PermissionStatusSource source) {
   // Explicitly switch to ensure that any new PermissionStatusSource values are
   // dealt with appropriately.
   switch (source) {
     case PermissionStatusSource::MULTIPLE_DISMISSALS:
       PermissionUmaUtil::RecordEmbargoPromptSuppression(
           PermissionEmbargoStatus::REPEATED_DISMISSALS);
       break;
+    case PermissionStatusSource::MULTIPLE_IGNORES:
+      PermissionUmaUtil::RecordEmbargoPromptSuppression(
+          PermissionEmbargoStatus::REPEATED_IGNORES);
+      break;
     case PermissionStatusSource::SAFE_BROWSING_BLACKLIST:
       PermissionUmaUtil::RecordEmbargoPromptSuppression(
           PermissionEmbargoStatus::PERMISSIONS_BLACKLISTING);
       break;
     case PermissionStatusSource::UNSPECIFIED:
     case PermissionStatusSource::KILL_SWITCH:
       // The permission wasn't under embargo, so don't record anything. We may
       // embargo it later.
       break;
   }
@@ skipping 356 matching lines @@
 
   const std::string rappor_metric = GetRapporMetric(permission, action);
   rappor::RapporServiceImpl* rappor_service =
       g_browser_process->rappor_service();
   if (!rappor_metric.empty() && rappor_service) {
     rappor_service->RecordSampleString(
         rappor_metric, rappor::LOW_FREQUENCY_ETLD_PLUS_ONE_RAPPOR_TYPE,
         rappor::GetDomainAndRegistrySampleFromGURL(requesting_origin));
   }
 }