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

Side by Side Diff: content/browser/download/download_manager_impl.h

Issue 9361046: Revert 121031 - Eliminate DownloadProgressUpdated. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 5
6 #ifndef CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_IMPL_H_ 6 #ifndef CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_IMPL_H_
7 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_IMPL_H_ 7 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_IMPL_H_
8 #pragma once 8 #pragma once
9 9
10 #include <map> 10 #include <map>
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 151
152 // Retrieves the download from the |download_id|. 152 // Retrieves the download from the |download_id|.
153 // Returns NULL if the download is not active. 153 // Returns NULL if the download is not active.
154 virtual content::DownloadItem* GetActiveDownload(int32 download_id) OVERRIDE; 154 virtual content::DownloadItem* GetActiveDownload(int32 download_id) OVERRIDE;
155 155
156 // Removes |download| from the active and in progress maps. 156 // Removes |download| from the active and in progress maps.
157 // Called when the download is cancelled or has an error. 157 // Called when the download is cancelled or has an error.
158 // Does nothing if the download is not in the history DB. 158 // Does nothing if the download is not in the history DB.
159 void RemoveFromActiveList(content::DownloadItem* download); 159 void RemoveFromActiveList(content::DownloadItem* download);
160 160
161 // Updates the delegate about the overall download progress.
162 void UpdateDownloadProgress();
163
161 // Inform observers that the model has changed. 164 // Inform observers that the model has changed.
162 void NotifyModelChanged(); 165 void NotifyModelChanged();
163 166
164 // Debugging routine to confirm relationship between below 167 // Debugging routine to confirm relationship between below
165 // containers; no-op if NDEBUG. 168 // containers; no-op if NDEBUG.
166 void AssertContainersConsistent() const; 169 void AssertContainersConsistent() const;
167 170
168 // Add a DownloadItem to history_downloads_. 171 // Add a DownloadItem to history_downloads_.
169 void AddDownloadItemToHistory(content::DownloadItem* item, int64 db_handle); 172 void AddDownloadItemToHistory(content::DownloadItem* item, int64 db_handle);
170 173
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
244 content::DownloadManagerDelegate* delegate_; 247 content::DownloadManagerDelegate* delegate_;
245 248
246 // TODO(rdsmith): Remove when http://crbug.com/85408 is fixed. 249 // TODO(rdsmith): Remove when http://crbug.com/85408 is fixed.
247 // For debugging only. 250 // For debugging only.
248 int64 largest_db_handle_in_history_; 251 int64 largest_db_handle_in_history_;
249 252
250 DISALLOW_COPY_AND_ASSIGN(DownloadManagerImpl); 253 DISALLOW_COPY_AND_ASSIGN(DownloadManagerImpl);
251 }; 254 };
252 255
253 #endif // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_IMPL_H_ 256 #endif // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_IMPL_H_
OLDNEW
« no previous file with comments | « chrome/browser/download/download_status_updater.cc ('k') | content/browser/download/download_manager_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698