Reporting: Plumb into RenderFrame via Mojo

content/renderer/render_frame_impl_browsertest.cc

 // Copyright 2015 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 <stdint.h>
 
 #include "base/command_line.h"
 #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"
 #include "content/common/frame_owner_properties.h"
 #include "content/common/renderer.mojom.h"
 #include "content/common/view_messages.h"
 #include "content/public/common/previews_state.h"
 #include "content/public/renderer/document_state.h"
 #include "content/public/test/frame_load_waiter.h"
 #include "content/public/test/render_view_test.h"
 #include "content/public/test/test_utils.h"
 #include "content/renderer/navigation_state_impl.h"
 #include "content/renderer/render_frame_impl.h"
 #include "content/renderer/render_view_impl.h"
 #include "content/test/fake_compositor_dependencies.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/public/platform/WebEffectiveConnectionType.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/platform/WebURL.h"
 #include "third_party/WebKit/public/platform/WebURLRequest.h"
 #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;
 
 namespace {
 const int32_t kSubframeRouteId = 20;
 const int32_t kSubframeWidgetRouteId = 21;
 const int32_t kFrameProxyRouteId = 22;
 }  // namespace
 
@@ skipping 496 matching lines @@
     request.SetURL(
         GURL(test.is_https ? "https://example.com" : "http://example.com"));
     request.SetPreviewsState(test.initial_request_previews_state);
 
     EXPECT_EQ(test.expected_should_use_client_lo_fi_for_request,
               frame()->ShouldUseClientLoFiForRequest(request))
         << (&test - tests);
   }
 }
 
+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