Reporting: Plumb into RenderFrame via Mojo

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index c5c6b04749a9de5743278470bc877171e85f9b35..3e80cec0c9cc31b8b9f54d1a30dc939b5324fded 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -44,6 +44,7 @@
 #include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/browser/media/media_interface_proxy.h"
 #include "content/browser/media/session/media_session_service_impl.h"
+#include "content/browser/net/reporting_service_proxy.h"
 #include "content/browser/permissions/permission_service_context.h"
 #include "content/browser/permissions/permission_service_impl.h"
 #include "content/browser/presentation/presentation_service_impl.h"
@@ -74,6 +75,7 @@
 #include "content/common/input_messages.h"
 #include "content/common/inter_process_time_ticks_converter.h"
 #include "content/common/navigation_params.h"
+#include "content/common/net/reporting.mojom.h"
 #include "content/common/render_message_filter.mojom.h"
 #include "content/common/renderer.mojom.h"
 #include "content/common/site_isolation_policy.h"
@@ -2857,6 +2859,9 @@ void RenderFrameHostImpl::RegisterMojoInterfaces() {
   GetInterfaceRegistry()->AddInterface(base::Bind(
       &KeyboardLockServiceImpl::CreateMojoService));
 
+  GetInterfaceRegistry()->AddInterface<mojom::ReportingServiceProxy>(base::Bind(

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

since this interface is per profile, it can be registered in
RenderProcessHostImpl instead of RenderFrameHost. that way if multiple frames in
a process use it, there's no need to keep fetching the interfaceptr from the
renderer (also less objects in general)

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

I can't -- the ReportingServiceProxy needs to know which SiteInstance (and
therefore which URLRequestContext) to route the report to, and render processes
(AFAIK) can host frames from different SiteInstances.

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

Do you mean StoragePartition instead of SiteInstance?

Each RenderProceeHost is associated with one StoragePartition from which one can
get a URLRequestContext (i.e.
RenderProcessHostImpl::GetStoragePartition()->GetURLRequestContext()->GetURLRequestContext()

+      &CreateReportingServiceProxy, base::RetainedRef(site_instance_)));
+
   GetContentClient()->browser()->ExposeInterfacesToFrame(GetInterfaceRegistry(),
                                                          this);
 }