[GCM] Add more UMA to GCM

google_apis/gcm/engine/checkin_request.cc

Index: google_apis/gcm/engine/checkin_request.cc
diff --git a/google_apis/gcm/engine/checkin_request.cc b/google_apis/gcm/engine/checkin_request.cc
index aa4665565f0075076a02643fdf8a0264bb514743..cbcd82bc1b9a63d593e694f574dca9635f66f897 100644
--- a/google_apis/gcm/engine/checkin_request.cc
+++ b/google_apis/gcm/engine/checkin_request.cc
@@ -144,6 +144,7 @@ void CheckinRequest::Start() {
   url_fetcher_->SetRequestContext(request_context_getter_);
   url_fetcher_->SetUploadData(kRequestContentType, upload_data);
   recorder_->RecordCheckinInitiated(request_info_.android_id);
+  request_start_time_ = base::TimeTicks::Now();
   url_fetcher_->Start();
 }
 
@@ -219,6 +220,10 @@ void CheckinRequest::OnURLFetchComplete(const net::URLFetcher* source) {
   }
 
   RecordCheckinStatusAndReportUMA(SUCCESS, recorder_, false);
+  UMA_HISTOGRAM_COUNTS("GCM.CheckinRetryNumber",

[fgorski, 2014/05/08 18:08:36]

Do we report try count if the checkin fails or do we attempt to check in
infinitely?

[jianli, 2014/05/08 18:40:07]

What we care about is the number of retries to do the checkin successfully. If
the checkin failed due to HTTP_BAD_REQUEST or HTTP_UNAUTHORIZED, it will be
reported by GCM.CheckinRequestStatus. Then we don't care about the number of
retries since no matter how many the checkin is retried, it should always fail.

+                       backoff_entry_.failure_count() + 1);
+  UMA_HISTOGRAM_TIMES("GCM.CheckinCompleteTime",
+                      base::TimeTicks::Now() - request_start_time_);
   callback_.Run(response_proto);
 }