Add a RAPPOR metric to track where users click to allow mixed scripts.

chrome/browser/content_settings/chrome_content_settings_utils.cc

 // Copyright (c) 2012 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/content_settings/chrome_content_settings_utils.h"
 
 #include "base/metrics/histogram.h"
+#include "chrome/browser/browser_process.h"
+#include "components/rappor/rappor_utils.h"
 
 namespace content_settings {
 
 void RecordMixedScriptAction(MixedScriptAction action) {
   UMA_HISTOGRAM_ENUMERATION("ContentSettings.MixedScript", action,
                             MIXED_SCRIPT_ACTION_COUNT);
 }
 
+void RecordMixedScriptActionWithRAPPOR(MixedScriptAction action,
+                                       const GURL& url) {
+  RecordMixedScriptAction(action);

[felt, 2015/06/12 00:18:25]

A second method seems fine, but can you avoid calling RecordMixedScriptAction
inside of it? It doesn't sound obvious that the WithRAPPOR metric would also do
the UMA. Instead, it's OK to have calls for both right next to each other.

[lgarron, 2015/06/12 00:30:18]

I chose "with RAPPOR" instead of "using RAPPOR", hoping it would be more clear.
But I don't feel strongly, so I've gone ahead and separated them.

+
+  std::string metric;
+  switch (action) {
+    case MIXED_SCRIPT_ACTION_DISPLAYED_SHIELD:
+      metric = "ContentSettings.MixedScript.DisplayedShield";
+      break;
+    case MIXED_SCRIPT_ACTION_CLICKED_ALLOW:
+      metric = "ContentSettings.MixedScript.UserClickedAllow";
+      break;
+    default:
+      NOTREACHED();
+  }
+
+  rappor::SampleDomainAndRegistryFromGURL(g_browser_process->rappor_service(),
+                                          metric,
+                                          url);
+}
+
 }  // namespace content_settings