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

Side by Side Diff: content/public/browser/download_manager.h

Issue 10912183: Remove DownloadManager::GetDownloadItem in favor of GetDownload() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 3 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 // The DownloadManager object manages the process of downloading, including 5 // The DownloadManager object manages the process of downloading, including
6 // updates to the history system and providing the information for displaying 6 // updates to the history system and providing the information for displaying
7 // the downloads view in the Destinations tab. There is one DownloadManager per 7 // the downloads view in the Destinations tab. There is one DownloadManager per
8 // active browser context in Chrome. 8 // active browser context in Chrome.
9 // 9 //
10 // Download observers: 10 // Download observers:
(...skipping 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
188 // The number of in progress (including paused) downloads. 188 // The number of in progress (including paused) downloads.
189 virtual int InProgressCount() const = 0; 189 virtual int InProgressCount() const = 0;
190 190
191 virtual BrowserContext* GetBrowserContext() const = 0; 191 virtual BrowserContext* GetBrowserContext() const = 0;
192 192
193 // Checks whether downloaded files still exist. Updates state of downloads 193 // Checks whether downloaded files still exist. Updates state of downloads
194 // that refer to removed files. The check runs in the background and may 194 // that refer to removed files. The check runs in the background and may
195 // finish asynchronously after this method returns. 195 // finish asynchronously after this method returns.
196 virtual void CheckForHistoryFilesRemoval() = 0; 196 virtual void CheckForHistoryFilesRemoval() = 0;
197 197
198 // Get the download item from the history map. Useful after the item has
199 // been removed from the active map, or was retrieved from the history DB.
200 virtual DownloadItem* GetDownloadItem(int id) = 0;
201
202 // Get the download item for |id| if present, no matter what type of download 198 // Get the download item for |id| if present, no matter what type of download
203 // it is or state it's in. 199 // it is or state it's in.
204 virtual DownloadItem* GetDownload(int id) = 0; 200 virtual DownloadItem* GetDownload(int id) = 0;
205 201
206 // Called when Save Page download is done. 202 // Called when Save Page download is done.
207 virtual void SavePageDownloadFinished(DownloadItem* download) = 0; 203 virtual void SavePageDownloadFinished(DownloadItem* download) = 0;
208 204
209 // Get the download item from the active map. Useful when the item is not
210 // yet in the history map.
211 virtual DownloadItem* GetActiveDownloadItem(int id) = 0;
212
213 protected: 205 protected:
214 virtual ~DownloadManager() {} 206 virtual ~DownloadManager() {}
215 207
216 private: 208 private:
217 friend class base::RefCountedThreadSafe<DownloadManager>; 209 friend class base::RefCountedThreadSafe<DownloadManager>;
218 }; 210 };
219 211
220 } // namespace content 212 } // namespace content
221 213
222 #endif // CONTENT_PUBLIC_BROWSER_DOWNLOAD_MANAGER_H_ 214 #endif // CONTENT_PUBLIC_BROWSER_DOWNLOAD_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698