Wire up CertificateReportingService to handle report uploads

chrome/browser/safe_browsing/certificate_reporting_metrics_provider.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/certificate_reporting_metrics_provider.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/safe_browsing/certificate_reporting_service.h"
 #include "chrome/browser/safe_browsing/certificate_reporting_service_factory.h"
 
 CertificateReportingMetricsProvider::CertificateReportingMetricsProvider() {}
 
 CertificateReportingMetricsProvider::~CertificateReportingMetricsProvider() {}
 
 void CertificateReportingMetricsProvider::ProvideGeneralMetrics(
-    metrics::ChromeUserMetricsExtension* unused) {}
+    metrics::ChromeUserMetricsExtension* unused) {
+  // When ProvideGeneralMetrics is called, this class is being asked to provide
+  // metrisc to the metrics service. It doesn't provide any metrics though,

[Ilya Sherman, 2016/12/23 00:56:48]

nit: s/metrisc/metrics

[meacer, 2016/12/27 18:56:43]

Done.

+  // instead it tells CertificateReportingService to upload any pending reports.
+  ProfileManager* profile_manager = g_browser_process->profile_manager();
+  if (!profile_manager)
+    return;
+
+  // Do not try to create profile here if it does not exist, because this method
+  // can be called during browser shutdown.
+  Profile* profile = profile_manager->GetProfileByPath(
+      profile_manager->GetLastUsedProfileDir(profile_manager->user_data_dir()));
+  if (!profile)
+    return;
+
+  CertificateReportingService* service =
+      CertificateReportingServiceFactory::GetForBrowserContext(profile);
+  if (service)
+    service->SendPending();
+}