Isolate initiation counts for downloads to their own histograms and improve

chrome/browser/download/download_throttling_resource_handler.cc

 // Copyright (c) 2012 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/download/download_throttling_resource_handler.h"
 
 #include "base/logging.h"
 #include "chrome/browser/download/download_request_limiter.h"
 #include "chrome/browser/download/download_util.h"
 #include "content/browser/renderer_host/resource_dispatcher_host.h"
@@ skipping 116 matching lines @@
 }
 
 void DownloadThrottlingResourceHandler::OnRequestClosed() {
   DCHECK(!request_closed_);
   if (request_allowed_)
     next_handler_->OnRequestClosed();
   request_closed_ = true;
 }
 
 void DownloadThrottlingResourceHandler::ContinueDownload(bool allow) {
-  download_util::RecordDownloadCount(
-        download_util::INITIATED_BY_NAVIGATION_COUNT);
   if (request_closed_)
     return;
 
   request_allowed_ = allow;
   if (allow) {
     // And let the request continue.
+    download_util::RecordDownloadSource(download_util::INITIATED_BY_NAVIGATION);
     host_->PauseRequest(render_process_host_id_, request_id_, false);
   } else {
+    download_util::RecordDownloadSource(download_util::BLOCKED_BY_THROTTLING);

[cbentzel, 2012/02/07 18:29:32]

I don't like having this as a separate source, as the source is ultimately the
same (navigation). Prefer to have a separate histogram for the throttling
behavior. 

[Randy Smith (Not in Mondays), 2012/02/07 19:28:14]

Moved into a "Counts" histogram at the chrome layer parallel to the same
histogram in content.

     host_->CancelRequest(render_process_host_id_, request_id_, false);
   }
 }