Add an IPC method for Blink to call RapporService::RecordSample()

content/renderer/renderer_blink_platform_impl.cc

Index: content/renderer/renderer_blink_platform_impl.cc
diff --git a/content/renderer/renderer_blink_platform_impl.cc b/content/renderer/renderer_blink_platform_impl.cc
index 2ac75e6a7b8d357af2e1b661b8cc4eac144cfff2..3d341b864338c21d3f78dc0d26e96db1ca3d8605 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -1009,6 +1009,13 @@ blink::WebString RendererBlinkPlatformImpl::convertIDNToUnicode(
 
 //------------------------------------------------------------------------------
 
+void RendererBlinkPlatformImpl::recordRappor(
+    const char* metric, const blink::WebString& sample) {

[timvolodine, 2015/03/25 18:56:19]

koji@: can we simply use blink::WebURL here? and then send the "sample" as a
GURL to the browser where GetDomainAndRegistrySampleFromGURL can be used on it.

In my patch I am using ContentBrowserClient::RecordURLMetric(metric, url) and I
am afraid your implementation is not exactly equivalent to this.

+  RenderThreadImpl::current()->RecordRappor(metric, sample.utf8());

[Steven Holte, 2015/03/25 17:57:26]

What's the format of the sample?  Is it a full host like
"codereview.chromium.org"?  Most places that use RAPPOR currently use
GetDomainAndRegistrySampleFromGURL to reduce samples down to just domain and
registry like "chromium.org".  If you include more this may make it a little
more difficult to analyze your results, since you will have a longer long tail
that will be lost in the noise, and will need a longer list of candidate host
names.

+}
+
+//------------------------------------------------------------------------------
+
 // static
 void RendererBlinkPlatformImpl::SetMockDeviceLightDataForTesting(double data) {
   g_test_device_light_data = data;