DownloadManagerDelegate::ShouldCompleteDownload(callback)

chrome/browser/chromeos/gdata/gdata_download_observer.h

 // 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_GDATA_GDATA_DOWNLOAD_OBSERVER_H__
 #define CHROME_BROWSER_CHROMEOS_GDATA_GDATA_DOWNLOAD_OBSERVER_H__
 #pragma once
 
 #include <map>
 
@@ skipping 26 matching lines @@
   // to external data in |download|.
   static void SetGDataPath(content::DownloadItem* download,
                            const FilePath& gdata_path);
 
   // Checks if there is a GData upload associated with |download|
   static bool IsGDataDownload(content::DownloadItem* download);
 
   // 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 |maybe_complete_download| is not null, then
+  // |maybe_complete_download| will be called on the UI thread when the download
+  // becomes ready to complete.  If you aren't careful to avoid setting

[Randy Smith (Not in Mondays), 2012/05/01 18:21:49]

I'd actually just say "If this call is made multiple times with the download not
ready to complete, only one callback will be generated, to the last value passed
as maybe_complete_download."  Either forbid or permit; there is no discourage
:-}.

[benjhayden, 2012/05/01 19:00:30]

Done.

+  // |maybe_complete_download| twice, then the last one wins.  You can't unset
+  // it once you set it.
+  static bool IsReadyToComplete(
+      content::DownloadItem* download,
+      const base::Closure& maybe_complete_download);
 
   // Returns the count of bytes confirmed as uploaded so far for |download|.
   static int64 GetUploadedBytes(content::DownloadItem* download);
 
   // Returns the progress of the upload of |download| as a percentage. If the
   // progress is unknown, returns -1.
   static int PercentComplete(content::DownloadItem* download);
 
   // Create a temporary file |gdata_tmp_download_path| in
   // |gdata_tmp_download_dir|. Must be called on a thread that allows file
@@ skipping 59 matching lines @@
   DownloadMap pending_downloads_;
 
   base::WeakPtrFactory<GDataDownloadObserver> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(GDataDownloadObserver);
 };
 
 }  // namespace gdata
 
 #endif  // CHROME_BROWSER_CHROMEOS_GDATA_GDATA_DOWNLOAD_OBSERVER_H__