Add Sync.Attachments.DownloadTotalTime.

sync/internal_api/attachments/attachment_downloader_impl.cc

 // Copyright 2014 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 "sync/internal_api/public/attachments/attachment_downloader_impl.h"
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/metrics/histogram.h"
+#include "base/metrics/histogram_macros.h"
 #include "base/metrics/sparse_histogram.h"
 #include "base/sys_byteorder.h"
+#include "base/time/time.h"
 #include "net/base/load_flags.h"
 #include "net/http/http_response_headers.h"
 #include "net/http/http_status_code.h"
 #include "net/http/http_util.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_status.h"
 #include "sync/internal_api/public/attachments/attachment_uploader_impl.h"
 #include "sync/internal_api/public/attachments/attachment_util.h"
 #include "sync/protocol/sync.pb.h"
 #include "url/gurl.h"
 
 namespace syncer {
 
 struct AttachmentDownloaderImpl::DownloadState {
  public:
   DownloadState(const AttachmentId& attachment_id,
                 const AttachmentUrl& attachment_url);
 
   AttachmentId attachment_id;
   AttachmentUrl attachment_url;
   // |access_token| needed to invalidate if downloading attachment fails with
   // HTTP_UNAUTHORIZED.
   std::string access_token;
   scoped_ptr<net::URLFetcher> url_fetcher;
   std::vector<DownloadCallback> user_callbacks;
+  base::TimeTicks start_time;
 };
 
 AttachmentDownloaderImpl::DownloadState::DownloadState(
     const AttachmentId& attachment_id,
     const AttachmentUrl& attachment_url)
     : attachment_id(attachment_id), attachment_url(attachment_url) {
 }
 
 AttachmentDownloaderImpl::AttachmentDownloaderImpl(
     const GURL& sync_service_url,
@@ skipping 54 matching lines @@
   access_token_request_.reset();
   StateList::const_iterator iter;
   // Start downloads for all download requests waiting for access token.
   for (iter = requests_waiting_for_access_token_.begin();
        iter != requests_waiting_for_access_token_.end();
        ++iter) {
     DownloadState* download_state = *iter;
     download_state->access_token = access_token;
     download_state->url_fetcher =
         CreateFetcher(download_state->attachment_url, access_token).Pass();
+    download_state->start_time = base::TimeTicks::Now();
     download_state->url_fetcher->Start();
   }
   requests_waiting_for_access_token_.clear();
 }
 
 void AttachmentDownloaderImpl::OnGetTokenFailure(
     const OAuth2TokenService::Request* request,
     const GoogleServiceAuthError& error) {
   DCHECK(CalledOnValidThread());
   DCHECK(request == access_token_request_.get());
@@ skipping 30 matching lines @@
 
   net::URLRequestStatus status = source->GetStatus();
   const int response_code = source->GetResponseCode();
   UMA_HISTOGRAM_SPARSE_SLOWLY("Sync.Attachments.DownloadResponseCode",
       status.is_success() ? response_code : status.error());
   if (response_code == net::HTTP_OK) {
     std::string data_as_string;
     source->GetResponseAsString(&data_as_string);
     attachment_data = base::RefCountedString::TakeString(&data_as_string);
 
+    UMA_HISTOGRAM_LONG_TIMES("Sync.Attachments.DownloadTotalTime",
+        base::TimeTicks::Now() - download_state.start_time);
+
     attachment_crc32c = ComputeCrc32c(attachment_data);
     uint32_t crc32c_from_headers = 0;
     if (ExtractCrc32c(source->GetResponseHeaders(), &crc32c_from_headers) &&
         attachment_crc32c != crc32c_from_headers) {
       // Fail download only if there is useful crc32c in header and it doesn't
       // match data. All other cases are fine. When crc32c is not in headers
       // locally calculated one will be stored and used for further checks.
       result = DOWNLOAD_TRANSIENT_ERROR;
     } else {
       result = DOWNLOAD_SUCCESS;
@@ skipping 93 matching lines @@
 
   if (crc32c_raw.size() != sizeof(*crc32c))
     return false;
 
   *crc32c =
       base::NetToHost32(*reinterpret_cast<const uint32_t*>(crc32c_raw.c_str()));
   return true;
 }
 
 }  // namespace syncer