Reporting: Plumb into RenderFrame via Mojo

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index a6ebbfb0c343a12bc3ba105dfb671119b23b8f48..a3ca222a53512b4902569fe2bd83164c1e929ab2 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -64,6 +64,7 @@
 #include "content/common/frame_replication_state.h"
 #include "content/common/input_messages.h"
 #include "content/common/navigation_params.h"
+#include "content/common/net/reporting.mojom.h"
 #include "content/common/page_messages.h"
 #include "content/common/savable_subframe.h"
 #include "content/common/service_worker/service_worker_types.h"
@@ -1112,6 +1113,15 @@ blink::WebURL RenderFrameImpl::OverrideFlashEmbedWithHTML(
   return GetContentClient()->renderer()->OverrideFlashEmbedWithHTML(url);
 }
 
+void RenderFrameImpl::QueueReport(const GURL& url,
+                                  const std::string& group,
+                                  const std::string& type,
+                                  std::unique_ptr<base::Value> body) {
+  mojom::ReportingServiceProxyPtr service;
+  GetRemoteInterfaces()->GetInterface(&service);
+  service->QueueReport(url, group, type, std::move(body));

[jam, 2017/06/05 21:03:44]

blink can connect directly to mojo services, no need to add plumbing to content/
for this. as part of onion soup, content/renderer is meant to be removed by
moving code to blink/. so we shouldn't add more code to move for new features.

[Julia Tuttle, 2017/06/15 20:33:32]

Okay!

[jam, 2017/06/23 16:39:41]

note: the mojom should then move to third_party/WebKit/public/platform/ so that
it's accessible to blink

+}
+
 // RenderFrameImpl ----------------------------------------------------------
 RenderFrameImpl::RenderFrameImpl(const CreateParams& params)
     : frame_(NULL),