Fix Rappor stats reporting for MixedContent.

chrome/renderer/content_settings_observer.cc

Index: chrome/renderer/content_settings_observer.cc
diff --git a/chrome/renderer/content_settings_observer.cc b/chrome/renderer/content_settings_observer.cc
index 1e00b6cb13b413fc408f4680a4760943f5e11a2a..85b7be3d6f1563ab5e1e7a3cb91e7df17a29fd26 100644
--- a/chrome/renderer/content_settings_observer.cc
+++ b/chrome/renderer/content_settings_observer.cc
@@ -586,7 +586,7 @@ bool ContentSettingsObserver::allowRunningInsecureContent(
     SendInsecureContentSignal(INSECURE_CONTENT_RUN_SWF);
 
   if (!allow_running_insecure_content_ && !allowed_per_settings) {
-    DidBlockContentType(CONTENT_SETTINGS_TYPE_MIXEDSCRIPT, origin.host());
+    DidBlockContentType(CONTENT_SETTINGS_TYPE_MIXEDSCRIPT, origin.toString());

[lgarron, 2015/11/06 23:25:08]

This seems like a good semantic fix.

Were you able to to tell that there are no other places that depend on this
being the host?

LGTM otherwise.

[Jialiu Lin, 2015/11/07 00:23:43]

I traced this details field, it basically can contains 2 types of information,
plugin name or host/url of a mixed content page.  The later one only used for
report RAPPOR stats. 

So nothing depends on details being the host

     return false;
   }