Add permission reporter skeleton

chrome/browser/safe_browsing/permission_reporter.h

+// Copyright 2016 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.
+
+#ifndef CHROME_BROWSER_SAFE_BROWSING_PERMISSION_REPORTER_H_
+#define CHROME_BROWSER_SAFE_BROWSING_PERMISSION_REPORTER_H_
+
+#include <string>
+
+#include "chrome/browser/permissions/permission_uma_util.h"
+#include "net/url_request/certificate_report_sender.h"
+#include "url/gurl.h"
+
+namespace net {
+class URLRequestContext;
+}  // namespace net
+
+namespace safe_browsing {
+
+// Provides functionality for building and serializing reports about permissions
+// to a report collection server.
+class PermissionReporter {
+ public:
+  // Creates a permission reporter that will send permission reports to
+  // |upload_url|, using |request_context| as the context for the reports.
+  PermissionReporter(net::URLRequestContext* request_context,
+                     const GURL& upload_url);
+
+  virtual ~PermissionReporter();
+
+  // Sends a permission report to the report collection server. The
+  // |serialized_report| is expected to be a serialized protobuf of type
+  // PermissionReport defined in
+  // //src/chrome/common/safe_browsing/permission_report.proto
+  virtual void SendReport(const std::string& serialized_report);
+
+  // 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.
+  static std::string BuildReport(const GURL& origin,
+                                 content::PermissionType permission,
+                                 PermissionAction action);

[kcarattini, 2016/05/31 04:56:22]

Actually, you can add a final argument which is a pointer to a string so that
you can fill in the serialized report there. I think this is what the protocol
buffer library function does so this will save copying the string around.

[stefanocs, 2016/05/31 05:05:26]

Yes, I can do that. Do you think it should returns a boolean instead (whether
the serialization is successful or not)? It seems the serialization won't fail
though since it doesn't have required fields.

[kcarattini, 2016/05/31 07:02:28]

Sure you can return the return value of SerializeToString directly.

[stefanocs, 2016/05/31 07:06:07]

Done.

+
+ private:
+  std::unique_ptr<net::CertificateReportSender> permission_report_sender_;
+
+  const GURL upload_url_;
+
+  DISALLOW_COPY_AND_ASSIGN(PermissionReporter);
+};
+
+}  // namespace safe_browsing
+
+#endif  // CHROME_BROWSER_SAFE_BROWSING_PERMISSION_REPORTER_H_