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

Side by Side Diff: chrome/browser/download/download_service.h

Issue 230103002: [Downloads] Ask DownloadHistory if a download was from history. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_ 5 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_
6 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_ 6 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/callback_forward.h" 9 #include "base/callback_forward.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 // ChromeDownloadManagerDelegate may be the target of callbacks from 74 // ChromeDownloadManagerDelegate may be the target of callbacks from
75 // the history service/DB thread and must be kept alive for those 75 // the history service/DB thread and must be kept alive for those
76 // callbacks. 76 // callbacks.
77 scoped_ptr<ChromeDownloadManagerDelegate> manager_delegate_; 77 scoped_ptr<ChromeDownloadManagerDelegate> manager_delegate_;
78 78
79 scoped_ptr<DownloadHistory> download_history_; 79 scoped_ptr<DownloadHistory> download_history_;
80 80
81 // The UI controller is responsible for observing the download manager and 81 // The UI controller is responsible for observing the download manager and
82 // notifying the UI of any new downloads. Its lifetime matches that of the 82 // notifying the UI of any new downloads. Its lifetime matches that of the
83 // associated download manager. 83 // associated download manager.
84 // Note on destruction order: download_ui_ depends on download_history_ and
85 // should be destroyed before the latter.
84 scoped_ptr<DownloadUIController> download_ui_; 86 scoped_ptr<DownloadUIController> download_ui_;
85 87
86 // On Android, GET downloads are not handled by the DownloadManager. 88 // On Android, GET downloads are not handled by the DownloadManager.
87 // Once we have extensions on android, we probably need the EventRouter 89 // Once we have extensions on android, we probably need the EventRouter
88 // in ContentViewDownloadDelegate which knows about both GET and POST 90 // in ContentViewDownloadDelegate which knows about both GET and POST
89 // downloads. 91 // downloads.
90 #if !defined(OS_ANDROID) 92 #if !defined(OS_ANDROID)
91 // The ExtensionDownloadsEventRouter dispatches download creation, change, and 93 // The ExtensionDownloadsEventRouter dispatches download creation, change, and
92 // erase events to extensions. Like ChromeDownloadManagerDelegate, it's a 94 // erase events to extensions. Like ChromeDownloadManagerDelegate, it's a
93 // chrome-level concept and its lifetime should match DownloadManager. There 95 // chrome-level concept and its lifetime should match DownloadManager. There
94 // should be a separate EDER for on-record and off-record managers. 96 // should be a separate EDER for on-record and off-record managers.
95 // There does not appear to be a separate ExtensionSystem for on-record and 97 // There does not appear to be a separate ExtensionSystem for on-record and
96 // off-record profiles, so ExtensionSystem cannot own the EDER. 98 // off-record profiles, so ExtensionSystem cannot own the EDER.
97 scoped_ptr<ExtensionDownloadsEventRouter> extension_event_router_; 99 scoped_ptr<ExtensionDownloadsEventRouter> extension_event_router_;
98 #endif 100 #endif
99 101
100 DISALLOW_COPY_AND_ASSIGN(DownloadService); 102 DISALLOW_COPY_AND_ASSIGN(DownloadService);
101 }; 103 };
102 104
103 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_ 105 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698