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 1003 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) {

[Alexei Svitkine (slow), 2015/04/07 21:01:56]

Nit: 1 param per line when the first one wraps. I think "git cl format" can
format this correctly - consider using?

[kojii, 2015/04/08 07:21:23]

Done, didn't know "git cl format", very nice, thank you!

+  content::GetContentClient()->renderer()->RecordRappor(metric, sample.utf8());
+}
+
+void RendererBlinkPlatformImpl::recordRapporURL(
+  const char* metric, const blink::WebURL& url) {
+  content::GetContentClient()->renderer()->RecordRapporURL(metric, url);

[jam, 2015/04/07 20:57:44]

here and above, remove "content::"

[kojii, 2015/04/08 07:21:23]

Done.

+}
+
+//------------------------------------------------------------------------------
+
 // 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