Isolate initiation counts for downloads to their own histograms and improve

content/browser/tab_contents/tab_contents.cc

Index: content/browser/tab_contents/tab_contents.cc
diff --git a/content/browser/tab_contents/tab_contents.cc b/content/browser/tab_contents/tab_contents.cc
index aaacf1b106787c3094948bee4aaa90a9cce32459..bbc41e87a9389346e805d1fdea68e902544df0f1 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -1093,9 +1093,9 @@ bool TabContents::IsSavable() {
 void TabContents::OnSavePage() {
   // If we can not save the page, try to download it.
   if (!IsSavable()) {
+    download_stats::RecordDownloadSource(
+        download_stats::INITIATED_BY_SAVE_PACKAGE_ON_NON_HTML);
     SaveURL(GetURL(), GURL(), true);
-    download_stats::RecordDownloadCount(
-        download_stats::INITIATED_BY_SAVE_PACKAGE_FAILURE_COUNT);
     return;
   }
 
@@ -1571,6 +1571,8 @@ void TabContents::OnUpdateZoomLimits(int minimum_percent,
 }
 
 void TabContents::OnSaveURL(const GURL& url) {
+  download_stats::RecordDownloadSource(
+      download_stats::INITIATED_BY_PEPPER_SAVE);
   // Check if the URL to save matches the URL of the main frame. Since this
   // message originates from Pepper plugins, it may not be the case if the
   // plugin is an embedded element.