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

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

Issue 11028131: Shift passage of FileStream in downloads system to be by scoped_ptr<>. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Incorporated Al's comments. Created 8 years, 2 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_item_impl.cc ('k') | content/browser/download/download_manager_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/download/download_item_impl_unittest.cc
diff --git a/content/browser/download/download_item_impl_unittest.cc b/content/browser/download/download_item_impl_unittest.cc
index 7a186af7659e9d242a7d1985e78fc8fd8b211a68..c2e0b9116b83d0a190c8ed0fb378329b49a580fc 100644
--- a/content/browser/download/download_item_impl_unittest.cc
+++ b/content/browser/download/download_item_impl_unittest.cc
@@ -70,14 +70,14 @@ class MockRequestHandle : public DownloadRequestHandleInterface {
class MockDownloadFileFactory : public content::DownloadFileFactory {
public:
content::DownloadFile* CreateFile(
- DownloadCreateInfo* info,
+ scoped_ptr<DownloadCreateInfo> info,
scoped_ptr<content::ByteStreamReader> stream_reader,
DownloadManager* mgr,
bool calculate_hash,
const net::BoundNetLog& bound_net_log) {
return MockCreateFile(
- info, stream_reader.get(), info->request_handle, mgr, calculate_hash,
- bound_net_log);
+ info.get(), stream_reader.get(), info->request_handle, mgr,
+ calculate_hash, bound_net_log);
}
MOCK_METHOD6(MockCreateFile,
« no previous file with comments | « content/browser/download/download_item_impl.cc ('k') | content/browser/download/download_manager_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698