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

Unified Diff: chrome/browser/download/download_item_model.cc

Issue 2853363002: Rename the DownloadService to DownloadCoreService (Closed)
Patch Set: Created 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/download/download_item_model.cc
diff --git a/chrome/browser/download/download_item_model.cc b/chrome/browser/download/download_item_model.cc
index 503527a544c94f722da19fd83213b4a8fc2d0aff..adb8de5d437b569e24e1c66d979518ae329ea23d 100644
--- a/chrome/browser/download/download_item_model.cc
+++ b/chrome/browser/download/download_item_model.cc
@@ -14,10 +14,10 @@
#include "base/supports_user_data.h"
#include "base/time/time.h"
#include "chrome/browser/download/chrome_download_manager_delegate.h"
+#include "chrome/browser/download/download_core_service.h"
+#include "chrome/browser/download/download_core_service_factory.h"
#include "chrome/browser/download/download_crx_util.h"
#include "chrome/browser/download/download_history.h"
-#include "chrome/browser/download/download_service.h"
-#include "chrome/browser/download/download_service_factory.h"
#include "chrome/browser/download/download_stats.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/safe_browsing/download_feedback_service.h"
@@ -615,10 +615,11 @@ void DownloadItemModel::SetShouldShowInShelf(bool should_show) {
bool DownloadItemModel::ShouldNotifyUI() const {
Profile* profile =
Profile::FromBrowserContext(download_->GetBrowserContext());
- DownloadService* download_service =
- DownloadServiceFactory::GetForBrowserContext(profile);
+ DownloadCoreService* download_core_service =
+ DownloadCoreServiceFactory::GetForBrowserContext(profile);
DownloadHistory* download_history =
- (download_service ? download_service->GetDownloadHistory() : NULL);
+ (download_core_service ? download_core_service->GetDownloadHistory()
+ : nullptr);
// The browser is only interested in new downloads. Ones that were restored
// from history are not displayed on the shelf. The downloads page
@@ -754,14 +755,14 @@ base::string16 DownloadItemModel::GetInProgressStatusString() const {
}
void DownloadItemModel::OpenUsingPlatformHandler() {
- DownloadService* download_service =
- DownloadServiceFactory::GetForBrowserContext(
+ DownloadCoreService* download_core_service =
+ DownloadCoreServiceFactory::GetForBrowserContext(
download_->GetBrowserContext());
- if (!download_service)
+ if (!download_core_service)
return;
ChromeDownloadManagerDelegate* delegate =
- download_service->GetDownloadManagerDelegate();
+ download_core_service->GetDownloadManagerDelegate();
if (!delegate)
return;
delegate->OpenDownloadUsingPlatformHandler(download_);

Powered by Google App Engine
This is Rietveld 408576698