diff options
| author | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-12 02:25:13 +0000 |
|---|---|---|
| committer | avi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-12 02:25:13 +0000 |
| commit | 27ee16ff2b9370d70fb93233b73ec01856f08017 (patch) | |
| tree | 0ee062b116b1b7ab29d9546b37a2ec3cc461e3b5 /chrome/browser/download/download_status_updater.h | |
| parent | d2a0e1ab98ab2e01fdb2cb77387e917c1d57b4c8 (diff) | |
| download | chromium_src-27ee16ff2b9370d70fb93233b73ec01856f08017.zip chromium_src-27ee16ff2b9370d70fb93233b73ec01856f08017.tar.gz chromium_src-27ee16ff2b9370d70fb93233b73ec01856f08017.tar.bz2 | |
Unify and clean up system download notifications, add Mountain Lion support.
BUG=138962
TEST=as in bug
Review URL: https://chromiumcodereview.appspot.com/10827207
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151195 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/download/download_status_updater.h')
| -rw-r--r-- | chrome/browser/download/download_status_updater.h | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/chrome/browser/download/download_status_updater.h b/chrome/browser/download/download_status_updater.h index 252be03..75218c5 100644 --- a/chrome/browser/download/download_status_updater.h +++ b/chrome/browser/download/download_status_updater.h @@ -42,12 +42,16 @@ class DownloadStatusUpdater virtual void OnDownloadDestroyed(content::DownloadItem* download) OVERRIDE; protected: - // Update the app icon. Virtual to be overridable for testing. - virtual void UpdateAppIconDownloadProgress(); + // Platform-specific function to update the platform UI for download progress. + // |download| is the download item that changed. Implementations should not + // hold the value of |download| as it is not guaranteed to remain valid. + // Virtual to be overridable for testing. + virtual void UpdateAppIconDownloadProgress(content::DownloadItem* download); private: - // Update the internal state tracking an item. - void UpdateItem(content::DownloadItem* download); + // Update the internal state tracking an item. Returns true if the item was + // added to the set of tracked items. + bool UpdateItem(content::DownloadItem* download); std::set<content::DownloadManager*> managers_; std::set<content::DownloadItem*> items_; |