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

Unified Diff: chrome/browser/automation/testing_automation_provider.cc

Issue 8503018: Split DownloadItem into an ABC, an Impl, and a Mock. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: merge Created 9 years, 1 month 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/automation/testing_automation_provider.cc
diff --git a/chrome/browser/automation/testing_automation_provider.cc b/chrome/browser/automation/testing_automation_provider.cc
index d47a5ec2629bf3b19c08d482e13104cfc63268a5..1b2b47ab2d351e7df892c7c758608fd763f43600 100644
--- a/chrome/browser/automation/testing_automation_provider.cc
+++ b/chrome/browser/automation/testing_automation_provider.cc
@@ -3127,7 +3127,7 @@ DownloadItem* GetDownloadItemFromId(int id, DownloadManager* download_manager) {
it != downloads.end();
it++) {
DownloadItem* curr_item = *it;
- if (curr_item->id() == id) {
+ if (curr_item->GetId() == id) {
selected_item = curr_item;
break;
}
@@ -3173,7 +3173,7 @@ void TestingAutomationProvider::PerformActionOnDownload(
selected_item->OpenDownload();
} else if (action == "toggle_open_files_like_this") {
DownloadPrefs* prefs =
- DownloadPrefs::FromDownloadManager(selected_item->download_manager());
+ DownloadPrefs::FromDownloadManager(selected_item->GetDownloadManager());
FilePath path = selected_item->GetUserVerifiedFilePath();
if (!selected_item->ShouldOpenFileBasedOnExtension())
prefs->EnableAutoOpenBasedOnExtension(path);

Powered by Google App Engine
This is Rietveld 408576698