Added member data to classes to support download resumption.

content/browser/renderer_host/resource_dispatcher_host_unittest.cc

 // 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.
 
 #include "content/browser/renderer_host/resource_dispatcher_host.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/file_path.h"
@@ skipping 15 matching lines @@
 #include "content/common/view_messages.h"
 #include "net/base/net_errors.h"
 #include "net/base/upload_data.h"
 #include "net/http/http_util.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_job.h"
 #include "net/url_request/url_request_test_job.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webkit/appcache/appcache_interfaces.h"
 
+class DownloadManager;

[Randy Smith (Not in Mondays), 2011/11/23 19:25:13]

I'm still confused as to what change in this CL made this addition to
resource_dispatcher_host_unittest.cc necessary?  (Assuming there is one, but
wanting to understand.)

[ahendrickson, 2011/12/20 00:04:14]

I removed:

#include "download_file.h"

from download_resource_handler.h, which was included in
resource_dispatcher_host.h (above).

download_file.h had a forward declaration of DownloadManager, which is used in
this file.

+
 using content::BrowserThread;
 using content::BrowserThreadImpl;
 
 // TODO(eroman): Write unit tests for SafeBrowsing that exercise
 //               SafeBrowsingResourceHandler.
 
 namespace {
 
 // Returns the resource response header structure for this request.
 void GetResponseHead(const std::vector<IPC::Message>& messages,
@@ skipping 1212 matching lines @@
   net::URLRequestStatus status;
 
   void* iter = NULL;
   EXPECT_TRUE(IPC::ReadParam(&msgs[0][0], &iter, &request_id));
   EXPECT_TRUE(IPC::ReadParam(&msgs[0][0], &iter, &status));
 
   EXPECT_EQ(1, request_id);
   EXPECT_EQ(net::URLRequestStatus::FAILED, status.status());
   EXPECT_EQ(net::ERR_UNKNOWN_URL_SCHEME, status.error());
 }