Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1920)

Unified Diff: chrome/browser/chromeos/gdata/gdata_download_observer.h

Issue 10263019: DownloadManagerDelegate::ShouldCompleteDownload(callback) (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: merge Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/chromeos/gdata/gdata_download_observer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/gdata/gdata_download_observer.h
diff --git a/chrome/browser/chromeos/gdata/gdata_download_observer.h b/chrome/browser/chromeos/gdata/gdata_download_observer.h
index 7c69877ffbbee78e0d24e48b6e8c6ad8e9c1b696..5b8342df12c87185cd5d59c978bb13e930f68425 100644
--- a/chrome/browser/chromeos/gdata/gdata_download_observer.h
+++ b/chrome/browser/chromeos/gdata/gdata_download_observer.h
@@ -44,8 +44,15 @@ class GDataDownloadObserver : public content::DownloadManager::Observer,
// Checks if |download| is ready to complete. Returns true if |download| has
// no GData upload associated with it or if the GData upload has already
// completed. This method is called by the ChromeDownloadManagerDelegate to
- // check if the download is ready to complete.
- static bool IsReadyToComplete(content::DownloadItem* download);
+ // check if the download is ready to complete. If the download is not yet
+ // ready to complete and |complete_callback| is not null, then
+ // |complete_callback| will be called on the UI thread when the download
+ // becomes ready to complete. If this method is called multiple times with
+ // the download not ready to complete, only the last |complete_callback|
+ // passed to this method for |download| will be called.
+ static bool IsReadyToComplete(
+ content::DownloadItem* download,
+ const base::Closure& complete_callback);
// Returns the count of bytes confirmed as uploaded so far for |download|.
static int64 GetUploadedBytes(content::DownloadItem* download);
« no previous file with comments | « no previous file | chrome/browser/chromeos/gdata/gdata_download_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698