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

Unified Diff: content/browser/download/download_manager_impl_unittest.cc

Issue 1875033005: [Downloads/History] Add tab-url and tab-referrer-url to DownloadRow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@guid-fix
Patch Set: Update expectations for DownloadUIController test. Created 4 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
« no previous file with comments | « content/browser/download/download_manager_impl.cc ('k') | content/public/browser/download_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/download/download_manager_impl_unittest.cc
diff --git a/content/browser/download/download_manager_impl_unittest.cc b/content/browser/download/download_manager_impl_unittest.cc
index 36bc00fc3848332f3c550f5893c65bfe3dc76f13..dfc016211c3bffffcd6fde6448426de9b2197f4a 100644
--- a/content/browser/download/download_manager_impl_unittest.cc
+++ b/content/browser/download/download_manager_impl_unittest.cc
@@ -85,6 +85,8 @@ class MockDownloadItemImpl : public DownloadItemImpl {
base::FilePath(),
std::vector<GURL>(),
GURL(),
+ GURL(),
+ GURL(),
"application/octet-stream",
"application/octet-stream",
base::Time(),
@@ -249,6 +251,8 @@ class MockDownloadItemFactory
const base::FilePath& target_path,
const std::vector<GURL>& url_chain,
const GURL& referrer_url,
+ const GURL& tab_url,
+ const GURL& tab_referrer_url,
const std::string& mime_type,
const std::string& original_mime_type,
const base::Time& start_time,
@@ -318,6 +322,8 @@ DownloadItemImpl* MockDownloadItemFactory::CreatePersistedItem(
const base::FilePath& target_path,
const std::vector<GURL>& url_chain,
const GURL& referrer_url,
+ const GURL& tab_url,
+ const GURL& tab_referrer_url,
const std::string& mime_type,
const std::string& original_mime_type,
const base::Time& start_time,
@@ -753,6 +759,8 @@ TEST_F(DownloadManagerTest, GetDownloadByGuid) {
base::FilePath(),
std::vector<GURL>(),
GURL("http://example.com/a"),
+ GURL("http://example.com/a"),
+ GURL("http://example.com/a"),
"application/octet-stream",
"application/octet-stream",
base::Time::Now(),
« no previous file with comments | « content/browser/download/download_manager_impl.cc ('k') | content/public/browser/download_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698