Add permission reporter skeleton

chrome/browser/safe_browsing/permission_reporter.cc

+// 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.
+
+#include "chrome/browser/safe_browsing/permission_reporter.h"
+#include "chrome/common/safe_browsing/permission_report.pb.h"
+
+using content::PermissionType;
+
+namespace safe_browsing {
+
+namespace {
+// URL to upload permission action reports.
+static const char kPermissionActionReportingUploadUrl[] =
+    "http://safebrowsing.googleusercontent.com/safebrowsing/clientreport/"
+    "permission-action";
+}  // namespace
+
+PermissionReporter::PermissionReporter(net::URLRequestContext* request_context)
+    : upload_url_(GURL(kPermissionActionReportingUploadUrl)) {}

[raymes, 2016/06/01 06:45:44]

nit: I think you should be able to omit the "GURL(" here, the constructor should
be called implicitly.

[stefanocs, 2016/06/01 07:25:02]

Done.

+
+PermissionReporter::~PermissionReporter() {}
+
+void PermissionReporter::SendReport(const std::string& serialized_report) {
+  // TODO(stefanocs): Implement SendReport function.
+}
+

[raymes, 2016/06/01 06:45:44]

nit: // static

[stefanocs, 2016/06/01 07:25:02]

I think we can't mark this as static here.

[kcarattini, 2016/06/01 12:25:49]

What he means is to put a comment above the function. That's how we mark static
functions in cc files.

[stefanocs, 2016/06/02 00:24:32]

Done.

+bool PermissionReporter::BuildReport(const GURL& origin,
+                                     PermissionType permission,
+                                     PermissionAction action,
+                                     std::string* output) {
+  // TODO(stefanocs): Implement BuildReport function.
+  return true;
+}
+
+}  // namespace safe_browsing