Reporting: Add histograms.

content/browser/net/reporting_service_proxy.cc

 // Copyright 2017 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/browser/net/reporting_service_proxy.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
 #include "content/common/net/reporting.mojom.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/storage_partition.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
+#include "net/reporting/reporting_report.h"
 #include "net/reporting/reporting_service.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "url/gurl.h"
 
 namespace content {
 
 namespace {
 
 std::string ReportTypeEnumToString(mojom::ReportingReportType type) {
@@ skipping 10 matching lines @@
 }
 
 void QueueReportOnIOThread(
     scoped_refptr<net::URLRequestContextGetter> request_context_getter,
     const GURL& url,
     const std::string& group,
     const std::string& type,
     std::unique_ptr<const base::Value> body) {
   net::URLRequestContext* request_context =
       request_context_getter->GetURLRequestContext();
-  if (!request_context)
+  if (!request_context) {
+    net::ReportingReport::RecordReportDiscardedForNoURLRequestContext();
     return;
+  }
 
   net::ReportingService* reporting_service =
       request_context->reporting_service();
-  if (!reporting_service)
+  if (!reporting_service) {
+    net::ReportingReport::RecordReportDiscardedForNoReportingService();
     return;
+  }
 
   // TODO(juliatuttle): Get rid of this copy.
   reporting_service->QueueReport(url, group, type, body->CreateDeepCopy());
 }
 
 class ReportingServiceProxyImpl : public mojom::ReportingServiceProxy {
  public:
   ReportingServiceProxyImpl(
       scoped_refptr<net::URLRequestContextGetter> request_context_getter)
       : request_context_getter_(std::move(request_context_getter)) {}
 
   // mojom::ReportingServiceProxy:
   void QueueReport(const GURL& url,
                    const std::string& group,
                    mojom::ReportingReportType type_enum,
                    std::unique_ptr<base::Value> body) override {
     std::string type_string = ReportTypeEnumToString(type_enum);
-    if (type_string.empty())
+    if (type_string.empty()) {
+      net::ReportingReport::RecordReportDiscardedForInvalidTypeFromRenderer();
       return;
+    }
 
     // TODO(juliatuttle): Get rid of this intermediate step.
     const base::Value* raw_const_body = body.release();
 
     request_context_getter_->GetNetworkTaskRunner()->PostTask(
         FROM_HERE,
         base::BindOnce(&QueueReportOnIOThread, request_context_getter_, url,
                        group, type_string, base::WrapUnique(raw_const_body)));
   }
 
@@ skipping 13 matching lines @@
       BrowserContext::GetStoragePartition(browser_context, site_instance);
   // TODO(juliatuttle): Make sure making our own scoped_refptr is okay.
   scoped_refptr<net::URLRequestContextGetter> request_context_getter(
       storage_partition->GetURLRequestContext());
   mojo::MakeStrongBinding(
       base::MakeUnique<ReportingServiceProxyImpl>(request_context_getter),
       std::move(request));
 }
 
 }  // namespace content