Add UMA to track permission changes from the content settings menu

chrome/browser/android/preferences/website_preference_bridge.cc

Index: chrome/browser/android/preferences/website_preference_bridge.cc
diff --git a/chrome/browser/android/preferences/website_preference_bridge.cc b/chrome/browser/android/preferences/website_preference_bridge.cc
index 64bf3e8909d1c8c6b54a6a3d23345be521fb1c64..1735445b408c17738497631c06b6bb6ebe58cd82 100644
--- a/chrome/browser/android/preferences/website_preference_bridge.cc
+++ b/chrome/browser/android/preferences/website_preference_bridge.cc
@@ -10,7 +10,7 @@
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/files/file_path.h"
-#include "base/strings/string_util.h"
+#include "base/metrics/histogram.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browsing_data/browsing_data_local_storage_helper.h"
 #include "chrome/browser/browsing_data/cookies_tree_model.h"
@@ -34,6 +34,37 @@ using base::android::ScopedJavaGlobalRef;
 using base::android::ScopedJavaLocalRef;
 using content::BrowserThread;
 
+namespace {
+
+static void LogPermissionChange(ContentSettingsType type,
+                                ContentSetting setting) {
+  ContentSettingsTypeHistogram histogram_value =
+      ContentSettingTypeToHistogramValue(type);
+  DCHECK_NE(histogram_value, CONTENT_SETTINGS_TYPE_HISTOGRAM_INVALID)
+      << "Invalid content setting type specified.";
+  UMA_HISTOGRAM_ENUMERATION("WebsiteSettings.Menu.PermissionChanged",
+                            histogram_value,
+                            CONTENT_SETTINGS_HISTOGRAM_NUM_TYPES);
+
+  if (setting == ContentSetting::CONTENT_SETTING_ALLOW) {
+    UMA_HISTOGRAM_ENUMERATION("WebsiteSettings.Menu.PermissionChanged.Allowed",
+                              histogram_value,
+                              CONTENT_SETTINGS_HISTOGRAM_NUM_TYPES);
+  } else if (setting == ContentSetting::CONTENT_SETTING_BLOCK) {
+    UMA_HISTOGRAM_ENUMERATION("WebsiteSettings.Menu.PermissionChanged.Blocked",
+                              histogram_value,
+                              CONTENT_SETTINGS_HISTOGRAM_NUM_TYPES);
+  } else if (setting == ContentSetting::CONTENT_SETTING_DEFAULT ||
+             setting == ContentSetting::CONTENT_SETTING_ASK) {
+    UMA_HISTOGRAM_ENUMERATION("WebsiteSettings.Menu.PermissionChanged.Ask",

[Finnur, 2015/02/23 14:15:27]

'Ask' is, as I recall, not something we can set for individual origins (it is
only available for global defaults), so I'm surprised to see CONTENT_SETTING_ASK
handled here. In what cases do you need to handle 'Ask' (for origins
specifically)?

I would like to think of this in terms of Allowed/Block/Reset to default, which
better reflects behind the scenes what happens (and is more future-proof, if we
change the global default for a category to be something other than 'Ask').

Therefore, I'd remove line 58 and change the UMA histogram name to be
WebsiteSettings.Menu.PermissionChanged.Reset, or something to that effect.

[Miguel Garcia, 2015/02/23 14:35:21]

Done.

+                              histogram_value,
+                              CONTENT_SETTINGS_HISTOGRAM_NUM_TYPES);
+  } else {
+    NOTREACHED();
+  }
+}
+}  // namespace
+
 static HostContentSettingsMap* GetHostContentSettingsMap() {
   Profile* profile = ProfileManager::GetActiveUserProfile();
   return profile->GetHostContentSettingsMap();
@@ -152,6 +183,7 @@ static void SetSettingForOrigin(JNIEnv* env,
       content_type,
       std::string(),
       setting);
+  LogPermissionChange(content_type, setting);
 }
 
 static void GetGeolocationOrigins(JNIEnv* env,
@@ -229,7 +261,7 @@ static void SetPushNotificationSettingForOrigin(JNIEnv* env, jclass clazz,
   // permission types. See https://crbug.com/416894.
   Profile* profile = ProfileManager::GetActiveUserProfile();
   GURL url = GURL(ConvertJavaStringToUTF8(env, origin));
-
+  ContentSetting setting = CONTENT_SETTING_DEFAULT;
   switch (value) {
     case -1:
       DesktopNotificationProfileUtil::ClearSetting(
@@ -237,13 +269,16 @@ static void SetPushNotificationSettingForOrigin(JNIEnv* env, jclass clazz,
       break;
     case 1:
       DesktopNotificationProfileUtil::GrantPermission(profile, url);
+      setting = CONTENT_SETTING_ALLOW;
       break;
     case 2:
       DesktopNotificationProfileUtil::DenyPermission(profile, url);
+      setting = CONTENT_SETTING_BLOCK;
       break;
     default:
       NOTREACHED();
   }
+  LogPermissionChange(CONTENT_SETTINGS_TYPE_NOTIFICATIONS, setting);
 }
 
 static void GetVoiceAndVideoCaptureOrigins(JNIEnv* env,