Created an interface for DownloadFile, for use in unit tests.

content/browser/download/download_resource_handler.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.
 
 #ifndef CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_RESOURCE_HANDLER_H_
 #define CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_RESOURCE_HANDLER_H_
 #pragma once
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/timer.h"
 #include "content/browser/download/download_file.h"
 #include "content/browser/download/download_id.h"
+#include "content/browser/download/download_types.h"

[Randy Smith (Not in Mondays), 2011/11/10 21:58:43]

I wouldn't think you'd need this include in both .h and .cc file for
download_resource_handler.  Remove it from one of this?  (Preferably the .h if
you can.)

[ahendrickson, 2011/11/13 00:50:21]

Need it in the header file, as we're declaring a DownloadSaveInfo member
variable.

Done.

 #include "content/browser/renderer_host/global_request_id.h"
 #include "content/browser/renderer_host/resource_handler.h"
 
 class DownloadFileManager;
 class ResourceDispatcherHost;
 
 namespace content {
 class DownloadBuffer;
 }
 
@@ skipping 81 matching lines @@
   base::TimeTicks download_start_time_;  // used to collect stats.
   base::TimeTicks last_read_time_;        // used to collect stats.
   size_t last_buffer_size_;               // used to collect stats.
   static const int kReadBufSize = 32768;  // bytes
   static const int kThrottleTimeMs = 200;  // milliseconds
 
   DISALLOW_COPY_AND_ASSIGN(DownloadResourceHandler);
 };
 
 #endif  // CONTENT_BROWSER_DOWNLOAD_DOWNLOAD_RESOURCE_HANDLER_H_