Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(280)

Unified Diff: components/metrics/reporting_service.cc

Issue 2774503002: Track network stack error codes from UMA and UKM (Closed)
Patch Set: Diffbase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/metrics/reporting_service.cc
diff --git a/components/metrics/reporting_service.cc b/components/metrics/reporting_service.cc
index fcff1f7f35719901baae1711ce303b4c57408182..4b1515e8b82e3fb00f966a9b7745c465be24f6cd 100644
--- a/components/metrics/reporting_service.cc
+++ b/components/metrics/reporting_service.cc
@@ -154,7 +154,7 @@ void ReportingService::SendStagedLog() {
log_uploader_->UploadLog(log_store()->staged_log(), hash);
}
-void ReportingService::OnLogUploadComplete(int response_code) {
+void ReportingService::OnLogUploadComplete(int response_code, int error_code) {
DVLOG(1) << "OnLogUploadComplete:" << response_code;
DCHECK(thread_checker_.CalledOnValidThread());
DCHECK(log_upload_in_progress_);
@@ -162,6 +162,8 @@ void ReportingService::OnLogUploadComplete(int response_code) {
// Log a histogram to track response success vs. failure rates.
LogResponseCode(response_code);
+ if (error_code)
+ LogErrorCode(error_code);
bool upload_succeeded = response_code == 200;

Powered by Google App Engine
This is Rietveld 408576698