Add report_id to ErrorReporter interface.

chrome/browser/ssl/certificate_reporting_test_utils.cc

 // Copyright 2015 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/ssl/certificate_reporting_test_utils.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 71 matching lines @@
 
 MockErrorReporter::MockErrorReporter(
     net::URLRequestContext* request_context,
     const GURL& upload_url,
     net::ReportSender::CookiesPreference cookies_preference)
     : certificate_reporting::ErrorReporter(request_context,
                                            upload_url,
                                            cookies_preference) {}
 
 void MockErrorReporter::SendExtendedReportingReport(
-    const std::string& serialized_report) {
+    const std::string& serialized_report,
+    int report_id) {

[Jialiu Lin, 2016/11/14 17:20:38]

nit: maybe report_id_unused

[meacer, 2016/11/14 23:34:54]

I've changed the interface to not accept a report_id, so this is removed now.

   certificate_reporting::ErrorReport report;
   ASSERT_TRUE(report.InitializeFromString(serialized_report));
   latest_hostname_reported_ = report.hostname();
 }
 
 void CertificateReportingTest::SetUpMockReporter() {
   // Set up the mock reporter to track the hostnames that reports get
   // sent for. The request_context argument is null here
   // because the MockErrorReporter doesn't actually use a
   // request_context. (In order to pass a real request_context, the
@@ skipping 53 matching lines @@
     if (!base::StringToDouble(param, &sendingThreshold))
       return CERT_REPORT_NOT_EXPECTED;
 
     if (sendingThreshold == 1.0)
       return certificate_reporting_test_utils::CERT_REPORT_EXPECTED;
   }
   return certificate_reporting_test_utils::CERT_REPORT_NOT_EXPECTED;
 }
 
 }  // namespace certificate_reporting_test_utils