Adding WebContent-free Download

content/browser/download/download_item_impl_unittest.cc

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 786d90311049b1fac20367975848dbc48aed379b..6a3b38df9875329bde35bd259a5fea81cfe28aac 100644
--- a/content/browser/download/download_item_impl_unittest.cc
+++ b/content/browser/download/download_item_impl_unittest.cc
@@ -60,7 +60,9 @@ class MockDelegate : public DownloadItemImplDelegate {
   MOCK_METHOD2(MockResumeInterruptedDownload,
                void(DownloadUrlParameters* params, uint32 id));
 
-  MOCK_CONST_METHOD0(GetBrowserContext, BrowserContext*());
+  BrowserContext* GetBrowserContext() const override {
+    return nullptr;

[asanka, 2015/12/08 16:58:32]

Why was this necessary?

When testing ResumeInterruptedDownload() we should ideally be verifying that
it's starting the correct URLRequest instead of aborting it by making
GetBrowserContext() return nullptr.

[svaldez, 2015/12/08 20:39:26]

Without either a WebContents or Browser/ResourceContext we can't create a
URLRequest so for tests that code through the ResumeInterruptedDownload
codepath, but aren't actually testing it by replacing it with
MockResumeInterruptedDownload we need to be able to make a GetBrowserContext
call.

+  }
   MOCK_METHOD1(UpdatePersistence, void(DownloadItemImpl*));
   MOCK_METHOD1(DownloadOpened, void(DownloadItemImpl*));
   MOCK_METHOD1(DownloadRemoved, void(DownloadItemImpl*));