Reporting: Plumb into RenderFrame via Mojo

content/renderer/render_frame_impl_browsertest.cc

Index: content/renderer/render_frame_impl_browsertest.cc
diff --git a/content/renderer/render_frame_impl_browsertest.cc b/content/renderer/render_frame_impl_browsertest.cc
index 56e3d95058bb9faf2d7a44a4c4fad7e574ad4014..e1fbfedb0ec2d21c5acb5cd06664045c7a13a6e0 100644
--- a/content/renderer/render_frame_impl_browsertest.cc
+++ b/content/renderer/render_frame_impl_browsertest.cc
@@ -8,6 +8,7 @@
 #include "base/debug/leak_annotations.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/values.h"
 #include "build/build_config.h"
 #include "content/child/web_url_loader_impl.h"
 #include "content/common/frame_messages.h"
@@ -31,6 +32,7 @@
 #include "third_party/WebKit/public/web/WebHistoryItem.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebView.h"
+#include "url/gurl.h"
 
 using blink::WebString;
 using blink::WebURLRequest;
@@ -547,4 +549,10 @@ TEST_F(RenderFrameImplTest, ShouldUseClientLoFiForRequest) {
   }
 }
 
+TEST_F(RenderFrameImplTest, ReportingSmokeTest) {
+  frame()->QueueReport(GURL("https://example.com/path"), "example-group",

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

what is this test testing, that no crashes occur?

seems like an end-to-end test would want to check the reporting service data
structures to check that it received it?

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

Yeah, this was just to check for crashes.

I'll hold out on an end-to-end test until we have a place that actually calls
it.

+                       "example-type",
+                       base::MakeUnique<base::DictionaryValue>());
+}
+
 }  // namespace content