DownloadManager::Observer::OnDownloadCreated

content/browser/download/download_manager_impl.cc

Index: content/browser/download/download_manager_impl.cc
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index 7570379eae814d624bfda65aab7759e8e965baef..08f736981573b04c9055c896d878c7cca74e0acc 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -534,6 +534,7 @@ net::BoundNetLog DownloadManagerImpl::CreateDownloadItem(
   downloads_[download->GetId()] = download;
   DCHECK(!ContainsKey(active_downloads_, download->GetId()));
   active_downloads_[download->GetId()] = download;
+  FOR_EACH_OBSERVER(Observer, observers_, OnDownloadCreated(this, download));
 
   return bound_net_log;
 }
@@ -563,6 +564,10 @@ DownloadItemImpl* DownloadManagerImpl::CreateSavePackageDownloadItem(
   new SavePageExternalData(download);
   DCHECK(SavePageExternalData::Get(download));
 
+  // TODO(benjhayden): Fire OnDownloadCreated for SavePackage downloads when
+  // we're comfortable with the user interacting with them.
+  // FOR_EACH_OBSERVER(Observer, observers_, OnDownloadCreated(this, download));
+
   // Will notify the observer in the callback.
   if (delegate_)
     delegate_->AddItemToPersistentStore(download);
@@ -947,6 +952,7 @@ void DownloadManagerImpl::OnPersistentStoreQueryComplete(
         this, GetNextId(), entries->at(i), bound_net_log);
     DCHECK(!ContainsKey(downloads_, download->GetId()));
     downloads_[download->GetId()] = download;
+    FOR_EACH_OBSERVER(Observer, observers_, OnDownloadCreated(this, download));
     VLOG(20) << __FUNCTION__ << "()" << i << ">"
              << " download = " << download->DebugString(true);
   }