Add rappor metrics for infobar/bubble operations for Geolocation and Notifications

chrome/browser/content_settings/permission_context_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 "base/metrics/histogram.h"
+#include "base/metrics/histogram_macros.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/content_settings/permission_context_uma_util.h"
+#include "components/rappor/rappor_utils.h"
 #include "content/public/browser/permission_type.h"
 #include "url/gurl.h"
 
 // UMA keys need to be statically initialized so plain function would not
 // work. Use a Macro instead.
 #define PERMISSION_ACTION_UMA(secure_origin,                                   \
-            permission, permission_secure, permission_insecure, action)        \
+        permission, permission_secure, permission_insecure, action)            \
     UMA_HISTOGRAM_ENUMERATION(                                                 \
         permission,                                                            \
         action,                                                                \
         PERMISSION_ACTION_NUM);                                                \
     if (secure_origin) {                                                       \
         UMA_HISTOGRAM_ENUMERATION(permission_secure,                           \
                                   action,                                      \
                                   PERMISSION_ACTION_NUM);                      \
     } else {                                                                   \
         UMA_HISTOGRAM_ENUMERATION(permission_insecure,                         \
                                   action,                                      \
                                   PERMISSION_ACTION_NUM);                      \
     }
 
+#define PERMISSION_ACTION_RAPPOR(gurl, granted, denied,                        \
+        dismissed, ignored, action)                                            \
+    switch (action) {                                                          \
+        case GRANTED:                                                          \
+          rappor::SampleDomainAndRegistryFromGURL(                             \
+              g_browser_process->rappor_service(), granted, gurl);             \
+          break;                                                               \
+        case DENIED:                                                           \
+          rappor::SampleDomainAndRegistryFromGURL(                             \
+              g_browser_process->rappor_service(), denied, gurl);              \
+          break;                                                               \
+        case DISMISSED:                                                        \
+          rappor::SampleDomainAndRegistryFromGURL(                             \
+              g_browser_process->rappor_service(), dismissed, gurl);           \
+          break;                                                               \
+        case IGNORED:                                                          \
+          rappor::SampleDomainAndRegistryFromGURL(                             \
+              g_browser_process->rappor_service(), ignored, gurl);             \
+          break;                                                               \
+         case PERMISSION_ACTION_NUM:                                           \
+           NOTREACHED();                                                       \
+           break;                                                              \
+    }
 
 namespace {
 
 // Enum for UMA purposes, make sure you update histograms.xml if you add new
 // permission actions. Never delete or reorder an entry; only add new entries
 // immediately before PERMISSION_NUM
 enum PermissionAction {
   GRANTED = 0,
   DENIED = 1,
   DISMISSED = 2,
   IGNORED = 3,
 
   // Always keep this at the end.
   PERMISSION_ACTION_NUM,
 };
 
 void RecordPermissionAction(
       ContentSettingsType permission,
       PermissionAction action,
-      bool secure_origin) {
+      const GURL& requesting_origin) {
+  bool secure_origin =  requesting_origin.SchemeIsSecure();
   switch (permission) {
       case CONTENT_SETTINGS_TYPE_GEOLOCATION:
         PERMISSION_ACTION_UMA(
             secure_origin,
             "ContentSettings.PermissionActions_Geolocation",
             "ContentSettings.PermissionActionsSecureOrigin_Geolocation",
             "ContentSettings.PermissionActionsInsecureOrigin_Geolocation",
             action);
+        PERMISSION_ACTION_RAPPOR(

[Steven Holte, 2015/03/18 20:39:20]

The rappor::Sample method doesn't initialize a static variable like the
histogram macros do, so it is probably better to just compute the metric name
here, rather than having the macro.

[Miguel Garcia, 2015/03/25 02:29:37]

Oh that's great news! Changed it to generate it dynamically.

+            requesting_origin,
+            "ContentSettings.PermissionActions_Geolocation.Granted.Url",
+            "ContentSettings.PermissionActions_Geolocation.Denied.Url",
+            "ContentSettings.PermissionActions_Geolocation.Dismissed.Url",
+            "ContentSettings.PermissionActions_Geolocation.Ignored.Url",
+            action);
         break;
       case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
         PERMISSION_ACTION_UMA(
             secure_origin,
             "ContentSettings.PermissionActions_Notifications",
             "ContentSettings.PermissionActionsSecureOrigin_Notifications",
             "ContentSettings.PermissionActionsInsecureOrigin_Notifications",
             action);
+        PERMISSION_ACTION_RAPPOR(
+            requesting_origin,
+            "ContentSettings.PermissionActions_Notifications.Granted.Url",
+            "ContentSettings.PermissionActions_Notifications.Denied.Url",
+            "ContentSettings.PermissionActions_Notifications.Dismissed.Url",
+            "ContentSettings.PermissionActions_Notifications.Ignored.Url",
+            action);
         break;
       case CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
         PERMISSION_ACTION_UMA(
             secure_origin,
             "ContentSettings.PermissionActions_MidiSysEx",
             "ContentSettings.PermissionActionsSecureOrigin_MidiSysEx",
             "ContentSettings.PermissionActionsInsecureOrigin_MidiSysEx",
             action);
         break;
       case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING:
@@ skipping 13 matching lines @@
             "ContentSettings.PermissionActionsInsecureOrigin_ProtectedMedia",
             action);
         break;
 #endif
       default:
         NOTREACHED() << "PERMISSION " << permission << " not accounted for";
     }
 }
 
 void RecordPermissionRequest(
-    ContentSettingsType permission, bool secure_origin) {
+    ContentSettingsType permission, const GURL& requesting_origin) {
+  bool secure_origin =  requesting_origin.SchemeIsSecure();
   content::PermissionType type;
   switch (permission) {
     case CONTENT_SETTINGS_TYPE_GEOLOCATION:
+      rappor::SampleDomainAndRegistryFromGURL(
+          g_browser_process->rappor_service(),
+          "ContentSettings.PermissionRequested.Geolocation.Url",
+          requesting_origin);
       type = content::PERMISSION_GEOLOCATION;
       break;
     case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
+      rappor::SampleDomainAndRegistryFromGURL(
+          g_browser_process->rappor_service(),
+          "ContentSettings.PermissionRequested.Notifications.Url",
+          requesting_origin);
       type = content::PERMISSION_NOTIFICATIONS;
       break;
     case CONTENT_SETTINGS_TYPE_MIDI_SYSEX:
       type = content::PERMISSION_MIDI_SYSEX;
       break;
     case CONTENT_SETTINGS_TYPE_PUSH_MESSAGING:
       type = content::PERMISSION_PUSH_MESSAGING;
       break;
 #if defined(OS_ANDROID) || defined(OS_CHROMEOS)
     case CONTENT_SETTINGS_TYPE_PROTECTED_MEDIA_IDENTIFIER:
@@ skipping 18 matching lines @@
         content::PERMISSION_NUM);
   }
 }
 
 } // namespace
 
 // Make sure you update histograms.xml permission histogram_suffix if you
 // add new permission
 void PermissionContextUmaUtil::PermissionRequested(
     ContentSettingsType permission, const GURL& requesting_origin) {
-  RecordPermissionRequest(permission, requesting_origin.SchemeIsSecure());
+  RecordPermissionRequest(permission, requesting_origin);
 }
 
 void PermissionContextUmaUtil::PermissionGranted(
     ContentSettingsType permission, const GURL& requesting_origin) {
-  RecordPermissionAction(permission, GRANTED,
-                         requesting_origin.SchemeIsSecure());
+  RecordPermissionAction(permission, GRANTED, requesting_origin);
 }
 
 void PermissionContextUmaUtil::PermissionDenied(
     ContentSettingsType permission, const GURL& requesting_origin) {
-  RecordPermissionAction(permission, DENIED,
-                         requesting_origin.SchemeIsSecure());
+  RecordPermissionAction(permission, DENIED, requesting_origin);
 }
 
 void PermissionContextUmaUtil::PermissionDismissed(
     ContentSettingsType permission, const GURL& requesting_origin) {
-  RecordPermissionAction(permission, DISMISSED,
-                         requesting_origin.SchemeIsSecure());
+  RecordPermissionAction(permission, DISMISSED, requesting_origin);
 }
 
 void PermissionContextUmaUtil::PermissionIgnored(
     ContentSettingsType permission, const GURL& requesting_origin) {
-  RecordPermissionAction(permission, IGNORED,
-                         requesting_origin.SchemeIsSecure());
+  RecordPermissionAction(permission, IGNORED, requesting_origin);
 }