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"
+
+using content::PermissionType;
+
+namespace safe_browsing {
+
+PermissionReporter::PermissionReporter(
+    net::URLRequestContext* request_context,

[stefanocs, 2016/05/31 03:34:22]

Should I move this up to the previous line too?

[kcarattini, 2016/05/31 03:53:36]

Sounds good but I think you need to fix the indentation.

[stefanocs, 2016/05/31 04:04:12]

Done.

+    const GURL& upload_url)
+    : permission_report_sender_(new net::CertificateReportSender(
+                                    request_context,
+                                    net::CertificateReportSender::
+                                    CookiesPreference::DO_NOT_SEND_COOKIES)),
+      upload_url_(upload_url) {
+  DCHECK(permission_report_sender_);
+  DCHECK(!upload_url.is_empty());
+}
+
+PermissionReporter::~PermissionReporter() {}
+
+void PermissionReporter::SendReport(const std::string& serialized_report) {
+  // TODO(stefanocs): Implement SendReport function

[kcarattini, 2016/05/31 03:13:37]

Add a full stop.

[stefanocs, 2016/05/31 03:34:22]

Done.

+}
+
+PermissionReport PermissionReporter::BuildReport(
+    const GURL& origin,

[stefanocs, 2016/05/31 03:34:22]

And move this up to the previous line?

[kcarattini, 2016/05/31 03:53:36]

sounds good. Thanks for catching this.

[stefanocs, 2016/05/31 04:04:12]

Done.

+    PermissionType permission,
+    PermissionAction action) {
+  // TODO(stefanocs): Implement BuildReport function

[kcarattini, 2016/05/31 03:13:37]

Add a full stop.

[stefanocs, 2016/05/31 03:34:22]

Done.

+  return PermissionReport();
+}
+
+} // namespace safe_browsing