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

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

Issue 9030032: Get rid of a bunch of tab_contents.h includes from chrome. These are all trivial changes to use W... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix cros Created 8 years, 12 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/chrome_download_manager_delegate.cc
===================================================================
--- chrome/browser/download/chrome_download_manager_delegate.cc (revision 116331)
+++ chrome/browser/download/chrome_download_manager_delegate.cc (working copy)
@@ -32,11 +32,11 @@
#include "chrome/common/extensions/user_script.h"
#include "chrome/common/pref_names.h"
#include "content/browser/download/download_status_updater.h"
-#include "content/browser/tab_contents/tab_contents.h"
#include "content/public/browser/download_file.h"
#include "content/public/browser/download_item.h"
#include "content/public/browser/download_manager.h"
#include "content/public/browser/notification_source.h"
+#include "content/public/browser/web_contents.h"
#include "grit/generated_resources.h"
#include "ui/base/l10n/l10n_util.h"
@@ -125,12 +125,12 @@
}
void ChromeDownloadManagerDelegate::ChooseDownloadPath(
- TabContents* tab_contents,
+ WebContents* web_contents,
const FilePath& suggested_path,
void* data) {
// Deletes itself.
new DownloadFilePicker(
- download_manager_, tab_contents, suggested_path, data);
+ download_manager_, web_contents, suggested_path, data);
}
bool ChromeDownloadManagerDelegate::OverrideIntermediatePath(
@@ -161,7 +161,7 @@
}
WebContents* ChromeDownloadManagerDelegate::
- GetAlternativeTabContentsToNotifyForDownload() {
+ GetAlternativeWebContentsToNotifyForDownload() {
// Start the download in the last active browser. This is not ideal but better
// than fully hiding the download from the user.
Browser* last_active = BrowserList::GetLastActiveWithProfile(profile_);
@@ -276,11 +276,11 @@
download_history_->RemoveEntriesBetween(remove_begin, remove_end);
}
-void ChromeDownloadManagerDelegate::GetSaveDir(TabContents* tab_contents,
+void ChromeDownloadManagerDelegate::GetSaveDir(WebContents* web_contents,
FilePath* website_save_dir,
FilePath* download_save_dir) {
Profile* profile =
- Profile::FromBrowserContext(tab_contents->GetBrowserContext());
+ Profile::FromBrowserContext(web_contents->GetBrowserContext());
PrefService* prefs = profile->GetPrefs();
// Check whether the preference has the preferred directory for saving file.
« no previous file with comments | « chrome/browser/download/chrome_download_manager_delegate.h ('k') | chrome/browser/download/download_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698