Add a feature to display a persistence toggle for permission prompts on Android.

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 67 matching lines @@
       action_str = "Ignored";
       break;
     case REVOKED:
       action_str = "Revoked";
       break;
     default:
       NOTREACHED();
       break;
   }
 
-  std::string permission_str =
-      PermissionUtil::GetPermissionString(permission);
+  std::string permission_str = PermissionUtil::GetPermissionString(permission);
   if (permission_str.empty())
     return "";
   return base::StringPrintf("ContentSettings.PermissionActions_%s.%s.Url",
                             permission_str.c_str(), action_str.c_str());
 }
 
 void RecordPermissionRequest(PermissionType permission,
                              const GURL& requesting_origin,
                              const GURL& embedding_origin,
                              Profile* profile) {
   rappor::RapporService* rappor_service = g_browser_process->rappor_service();
   if (rappor_service) {
     if (permission == PermissionType::GEOLOCATION) {
       // TODO(dominickn): remove this deprecated metric - crbug.com/605836.
       rappor::SampleDomainAndRegistryFromGURL(
-          rappor_service,
-          "ContentSettings.PermissionRequested.Geolocation.Url",
+          rappor_service, "ContentSettings.PermissionRequested.Geolocation.Url",
           requesting_origin);
       rappor_service->RecordSample(
           "ContentSettings.PermissionRequested.Geolocation.Url2",
           rappor::LOW_FREQUENCY_ETLD_PLUS_ONE_RAPPOR_TYPE,
           rappor::GetDomainAndRegistrySampleFromGURL(requesting_origin));
     } else if (permission == PermissionType::NOTIFICATIONS) {
       // TODO(dominickn): remove this deprecated metric - crbug.com/605836.
       rappor::SampleDomainAndRegistryFromGURL(
           rappor_service,
           "ContentSettings.PermissionRequested.Notifications.Url",
           requesting_origin);
       rappor_service->RecordSample(
           "ContentSettings.PermissionRequested.Notifications.Url2",
           rappor::LOW_FREQUENCY_ETLD_PLUS_ONE_RAPPOR_TYPE,
           rappor::GetDomainAndRegistrySampleFromGURL(requesting_origin));
     } else if (permission == PermissionType::MIDI ||
                permission == PermissionType::MIDI_SYSEX) {
       // TODO(dominickn): remove this deprecated metric - crbug.com/605836.
       rappor::SampleDomainAndRegistryFromGURL(
-          rappor_service,
-          "ContentSettings.PermissionRequested.Midi.Url",
+          rappor_service, "ContentSettings.PermissionRequested.Midi.Url",
           requesting_origin);
       rappor_service->RecordSample(
           "ContentSettings.PermissionRequested.Midi.Url2",
           rappor::LOW_FREQUENCY_ETLD_PLUS_ONE_RAPPOR_TYPE,
           rappor::GetDomainAndRegistrySampleFromGURL(requesting_origin));
     }
   }
 
   bool secure_origin = content::IsOriginSecure(requesting_origin);
   UMA_HISTOGRAM_ENUMERATION(
@@ skipping 211 matching lines @@
   DCHECK(!requests.empty());
   DCHECK(requests.size() == 1);
 
   PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied,
                              requests[0]->GetPermissionRequestType());
   PERMISSION_BUBBLE_GESTURE_TYPE_UMA(
       kPermissionsPromptDeniedGesture, kPermissionsPromptDeniedNoGesture,
       requests[0]->GetGestureType(), requests[0]->GetPermissionRequestType());
 }
 
+void PermissionUmaUtil::PermissionPromptGrantedWithPersistenceToggle(
+    content::PermissionType permission,
+    bool toggle_enabled) {
+  switch (permission) {
+    case PermissionType::GEOLOCATION:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Granted.Persisted.Geolocation",

[tsergeant, 2016/08/11 00:21:50]

Maybe change the histogram names to

Permissions.Prompt.Accepted.Persisted.*

to match the existing Permissions.Prompt.Accepted metrics?

(Although you do need to be careful about directly comparing with those, since
they're not collected on Android yet).

[dominickn, 2016/08/11 04:43:49]

Done.

+                            toggle_enabled);
+      break;
+    case PermissionType::NOTIFICATIONS:
+      UMA_HISTOGRAM_BOOLEAN(
+          "Permissions.Prompt.Granted.Persisted.Notifications", toggle_enabled);
+      break;
+    case PermissionType::MIDI_SYSEX:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Granted.Persisted.MidiSysEx",
+                            toggle_enabled);
+      break;
+    case PermissionType::PUSH_MESSAGING:
+      UMA_HISTOGRAM_BOOLEAN(
+          "Permissions.Prompt.Granted.Persisted.PushMessaging", toggle_enabled);
+      break;
+    case PermissionType::PROTECTED_MEDIA_IDENTIFIER:
+      UMA_HISTOGRAM_BOOLEAN(
+          "Permissions.Prompt.Granted.Persisted.ProtectedMedia",
+          toggle_enabled);
+      break;
+    case PermissionType::DURABLE_STORAGE:
+      UMA_HISTOGRAM_BOOLEAN(
+          "Permissions.Prompt.Granted.Persisted.DurableStorage",
+          toggle_enabled);
+      break;
+    case PermissionType::AUDIO_CAPTURE:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Granted.Persisted.AudioCapture",
+                            toggle_enabled);
+      break;
+    case PermissionType::VIDEO_CAPTURE:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Granted.Persisted.VideoCapture",
+                            toggle_enabled);
+      break;
+    // The user is not prompted for these permissions, thus there is no
+    // grant recorded for them.
+    case PermissionType::MIDI:
+    case PermissionType::BACKGROUND_SYNC:
+    case PermissionType::NUM:
+      NOTREACHED() << "PERMISSION "
+                   << PermissionUtil::GetPermissionString(permission)
+                   << " not accounted for";
+  }
+}
+
+void PermissionUmaUtil::PermissionPromptDeniedWithPersistenceToggle(
+    content::PermissionType permission,
+    bool toggle_enabled) {
+  switch (permission) {
+    case PermissionType::GEOLOCATION:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Denied.Persisted.Geolocation",
+                            toggle_enabled);
+      break;
+    case PermissionType::NOTIFICATIONS:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Denied.Persisted.Notifications",
+                            toggle_enabled);
+      break;
+    case PermissionType::MIDI_SYSEX:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Denied.Persisted.MidiSysEx",
+                            toggle_enabled);
+      break;
+    case PermissionType::PUSH_MESSAGING:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Denied.Persisted.PushMessaging",
+                            toggle_enabled);
+      break;
+    case PermissionType::PROTECTED_MEDIA_IDENTIFIER:
+      UMA_HISTOGRAM_BOOLEAN(
+          "Permissions.Prompt.Denied.Persisted.ProtectedMedia", toggle_enabled);
+      break;
+    case PermissionType::DURABLE_STORAGE:
+      UMA_HISTOGRAM_BOOLEAN(
+          "Permissions.Prompt.Denied.Persisted.DurableStorage", toggle_enabled);
+      break;
+    case PermissionType::AUDIO_CAPTURE:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Denied.Persisted.AudioCapture",
+                            toggle_enabled);
+      break;
+    case PermissionType::VIDEO_CAPTURE:
+      UMA_HISTOGRAM_BOOLEAN("Permissions.Prompt.Denied.Persisted.VideoCapture",
+                            toggle_enabled);
+      break;
+    // The user is not prompted for these permissions, thus there is no
+    // deny recorded for them.
+    case PermissionType::MIDI:
+    case PermissionType::BACKGROUND_SYNC:
+    case PermissionType::NUM:
+      NOTREACHED() << "PERMISSION "
+                   << PermissionUtil::GetPermissionString(permission)
+                   << " not accounted for";
+  }
+}
+
 bool PermissionUmaUtil::IsOptedIntoPermissionActionReporting(Profile* profile) {
   if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kEnablePermissionActionReporting))
     return false;
 
   DCHECK(profile);
   if (profile->GetProfileType() == Profile::INCOGNITO_PROFILE)
     return false;
   if (!profile->GetPrefs()->GetBoolean(prefs::kSafeBrowsingEnabled))
     return false;
@@ skipping 103 matching lines @@
   if (!deprecated_metric.empty() && rappor_service) {
     rappor::SampleDomainAndRegistryFromGURL(rappor_service, deprecated_metric,
                                             requesting_origin);
 
     std::string rappor_metric = deprecated_metric + "2";
     rappor_service->RecordSample(
         rappor_metric, rappor::LOW_FREQUENCY_ETLD_PLUS_ONE_RAPPOR_TYPE,
         rappor::GetDomainAndRegistrySampleFromGURL(requesting_origin));
   }
 }