Add implementation of PermissionReporter

chrome/browser/safe_browsing/permission_reporter.h

Index: chrome/browser/safe_browsing/permission_reporter.h
diff --git a/chrome/browser/safe_browsing/permission_reporter.h b/chrome/browser/safe_browsing/permission_reporter.h
index a226dcf330d1b008429fa653b5fd10baf550a485..523de8e57057aa67d749f31333c690896448f132 100644
--- a/chrome/browser/safe_browsing/permission_reporter.h
+++ b/chrome/browser/safe_browsing/permission_reporter.h
@@ -11,6 +11,7 @@
 #include "url/gurl.h"
 
 namespace net {
+class ReportSender;
 class URLRequestContext;
 }  // namespace net
 
@@ -37,6 +38,13 @@ class PermissionReporter {
                   content::PermissionType permission,
                   PermissionAction action);
 
+ private:
+  friend class PermissionReporterTest;

[raymes, 2016/06/09 06:03:47]

Since the only thing used in the test is the additional constructor now, I'm ok
with leaving that as public and not having a friend class (to keep the test
simple). But either way is ok.

[stefanocs, 2016/06/09 06:41:29]

Acknowledged.

+
+  // Used by tests. This constructor allows tests to have access to the
+  // ReportSender.
+  explicit PermissionReporter(std::unique_ptr<net::ReportSender> report_sender);
+
   // Builds and serializes a permission report with |origin| as the origin of
   // the site requesting permission, |permission| as the type of permission
   // requested, and |action| as the action taken. The serialized report is
@@ -47,7 +55,8 @@ class PermissionReporter {
                           PermissionAction action,
                           std::string* output);
 
- private:
+  std::unique_ptr<net::ReportSender> permission_report_sender_;
+
   DISALLOW_COPY_AND_ASSIGN(PermissionReporter);
 };