Split DownloadItem into an ABC, an Impl, and a Mock.

content/browser/download/download_manager.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // The DownloadManager object manages the process of downloading, including
 // updates to the history system and providing the information for displaying
 // the downloads view in the Destinations tab. There is one DownloadManager per
 // active browser context in Chrome.
 //
 // Download observers:
@@ skipping 41 matching lines @@
 #include "content/public/browser/browser_thread.h"
 #include "net/base/net_errors.h"
 
 class DownloadFileManager;
 class DownloadIdFactory;
 class DownloadRequestHandle;
 class DownloadStatusUpdater;
 class GURL;
 class TabContents;
 struct DownloadCreateInfo;
+struct DownloadPersistentStoreInfo;

[cbentzel, 2011/11/16 12:28:44]

Was this needed due to changes in download_item.h?

[benjhayden, 2011/11/18 18:59:34]

Huh, I guess I don't need that.

 struct DownloadSaveInfo;
 
 namespace content {
 class BrowserContext;
 class DownloadManagerDelegate;
 }
 
 // Browser's download manager: manages all downloads and destination view.
 class CONTENT_EXPORT DownloadManager
     : public base::RefCountedThreadSafe<
@@ skipping 344 matching lines @@
   DownloadIdFactory* id_factory_;
 
   // TODO(rdsmith): Remove when http://crbug.com/85408 is fixed.
   // For debugging only.
   int64 largest_db_handle_in_history_;
 
   DISALLOW_COPY_AND_ASSIGN(DownloadManager);
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_MANAGER_H_