Reland DownloadManager::GetAllDownloads actually does now

chrome/browser/chromeos/gdata/drive_download_observer.cc

Index: chrome/browser/chromeos/gdata/drive_download_observer.cc
diff --git a/chrome/browser/chromeos/gdata/drive_download_observer.cc b/chrome/browser/chromeos/gdata/drive_download_observer.cc
index b13fac055cbe9ac600891f5c1f5ffe2fbcbcb3d6..79e451b5ecf823b472f6de20bd05f6c36699f8ad 100644
--- a/chrome/browser/chromeos/gdata/drive_download_observer.cc
+++ b/chrome/browser/chromeos/gdata/drive_download_observer.cc
@@ -351,13 +351,15 @@ void DriveDownloadObserver::ModelChanged(DownloadManager* download_manager) {
 
   DownloadManager::DownloadVector downloads;
   // Drive downloads are considered temporary downloads.
-  download_manager->GetTemporaryDownloads(drive_tmp_download_path_,
-                                          &downloads);
+  download_manager->GetAllDownloads(&downloads);
   for (size_t i = 0; i < downloads.size(); ++i) {
     // Only accept downloads that have the Drive meta data associated with
     // them. Otherwise we might trip over non-Drive downloads being saved to
     // drive_tmp_download_path_.
-    if (IsDriveDownload(downloads[i]))
+    if (downloads[i]->IsTemporary() &&
+        (downloads[i]->GetTargetFilePath().DirName() ==
+         drive_tmp_download_path_) &&
+        IsDriveDownload(downloads[i]))
       OnDownloadUpdated(downloads[i]);
   }
 }
@@ -456,9 +458,11 @@ bool DriveDownloadObserver::ShouldUpload(DownloadItem* download) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
   // Upload if the item is in pending_downloads_,
+  // has a filename,
   // is complete or large enough to stream, and,
   // is not already being uploaded.
   return (pending_downloads_.count(download->GetId()) != 0) &&
+         !download->GetFullPath().empty() &&
          (download->AllDataSaved() ||
           download->GetReceivedBytes() > kStreamingFileSize) &&
          (GetUploadingUserData(download) == NULL);