Add UMA stats for the Website Setting UI

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 da5761620965b60c84a470032b77844f340e0280..744e4b37d888e6efbca3702b7f0d2936f171f8b4 100644
--- a/chrome/browser/ui/website_settings/website_settings.cc
+++ b/chrome/browser/ui/website_settings/website_settings.cc
@@ -9,6 +9,7 @@
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/metrics/histogram.h"
 #include "base/i18n/time_formatting.h"
 #include "base/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
@@ -31,6 +32,7 @@
 #include "chrome/common/content_settings_pattern.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/cert_store.h"
+#include "content/public/browser/user_metrics.h"
 #include "content/public/common/ssl_status.h"
 #include "content/public/common/url_constants.h"
 #include "grit/chromium_strings.h"
@@ -97,6 +99,12 @@ WebsiteSettings::WebsiteSettings(
   PresentSiteData();
   PresentSiteIdentity();
   PresentHistoryInfo(base::Time());
+
+  // Everytime the Website Settings UI is opend a |WebsiteSettings| object is

[Ben Goodger (Google), 2012/09/06 21:22:35]

Every time

[Ben Goodger (Google), 2012/09/06 21:22:35]

opened

[markusheintz_, 2012/09/07 13:31:35]

Done.

[markusheintz_, 2012/09/07 13:31:35]

Done.

+  // created. So this counter counts how ofter the Website Settings UI is

[Finnur, 2012/09/06 10:10:04]

s/counter counts/counts/

[markusheintz_, 2012/09/07 13:31:35]

Done.

+  // opend.

[Finnur, 2012/09/06 10:10:04]

You can probably move this up a line if you follow my comment two lines up (oh,
and fix the typo).

[markusheintz_, 2012/09/07 13:31:35]

Done.

+  content::RecordAction(
+      content::UserMetricsAction("WebsiteSettings_Opened"));
 }
 
 WebsiteSettings::~WebsiteSettings() {
@@ -104,6 +112,10 @@ WebsiteSettings::~WebsiteSettings() {
 
 void WebsiteSettings::OnSitePermissionChanged(ContentSettingsType type,
                                               ContentSetting setting) {
+  // Count how often a permission for a specific content type is changed using
+  // the Website Settings UI.
+  UMA_HISTOGRAM_COUNTS("WebsiteSettings.PermissionChanged", type);
+
   ContentSettingsPattern primary_pattern;
   ContentSettingsPattern secondary_pattern;
   switch (type) {