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

content/renderer/renderer_blink_platform_impl.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 "content/renderer/renderer_blink_platform_impl.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
@@ skipping 991 matching lines @@
 //------------------------------------------------------------------------------
 
 blink::WebString RendererBlinkPlatformImpl::convertIDNToUnicode(
     const blink::WebString& host,
     const blink::WebString& languages) {
   return net::IDNToUnicode(host.utf8(), languages.utf8());
 }
 
 //------------------------------------------------------------------------------
 
+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;
 }
 
 //------------------------------------------------------------------------------
 
 // static
 void RendererBlinkPlatformImpl::SetMockDeviceMotionDataForTesting(
     const blink::WebDeviceMotionData& data) {
@@ skipping 179 matching lines @@
 //------------------------------------------------------------------------------
 
 void RendererBlinkPlatformImpl::MockBatteryStatusChangedForTesting(
     const blink::WebBatteryStatus& status) {
   if (!g_test_battery_status_listener)
     return;
   g_test_battery_status_listener->updateBatteryStatus(status);
 }
 
 }  // namespace content