Track network stack error codes from UMA and UKM

components/metrics/metrics_reporting_service.cc

Index: components/metrics/metrics_reporting_service.cc
diff --git a/components/metrics/metrics_reporting_service.cc b/components/metrics/metrics_reporting_service.cc
index 116a67f190a4ebc1d04146386e750c3447e633eb..26db01cc50527f7499d9659dfc75eed42d5f1325 100644
--- a/components/metrics/metrics_reporting_service.cc
+++ b/components/metrics/metrics_reporting_service.cc
@@ -96,6 +96,10 @@ void MetricsReportingService::LogResponseCode(int response_code) {
                             NUM_RESPONSE_STATUSES);
 }
 
+void MetricsReportingService::LogErrorCode(int error_code) {
+  UMA_HISTOGRAM_SPARSE_SLOWLY("UMA.FailedUploadErrorCode", -error_code);

[Alexei Svitkine (slow), 2017/03/23 15:23:42]

I think error and response code should be logged in the same histogram, like we
do for Variations.SeedFetchResponseOrErrorCode.

We can deprecate UMA.UploadResponseStatus.Protobuf at the same time.

[Steven Holte, 2017/03/23 20:38:07]

Done.

+}
+
 void MetricsReportingService::LogSuccess(size_t log_size) {
   UMA_HISTOGRAM_COUNTS_10000("UMA.LogSize.OnSuccess", log_size / 1024);
 }