Change DownloadProcessHandle to be more of an encapsulated class.

chrome/browser/download/download_file.cc

Index: chrome/browser/download/download_file.cc
diff --git a/chrome/browser/download/download_file.cc b/chrome/browser/download/download_file.cc
index d1a3470c00a3d38c75b8fbb7426aff053693cb76..5860ee47380c03e625dd28bd6e9516635cea84fa 100644
--- a/chrome/browser/download/download_file.cc
+++ b/chrome/browser/download/download_file.cc
@@ -30,9 +30,8 @@ DownloadFile::~DownloadFile() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 }
 
-void DownloadFile::CancelDownloadRequest(ResourceDispatcherHost* rdh) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
-  download_util::CancelDownloadRequest(rdh, process_handle_);
+void DownloadFile::CancelDownloadRequest() {

[Paweł Hajdan Jr., 2011/06/04 08:58:51]

nit: Please add DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)).

[Randy Smith (Not in Mondays), 2011/06/07 22:41:11]

Done.

+  process_handle_.CancelRequest();
 }
 
 DownloadManager* DownloadFile::GetDownloadManager() {
@@ -43,14 +42,10 @@ DownloadManager* DownloadFile::GetDownloadManager() {
 std::string DownloadFile::DebugString() const {
   return base::StringPrintf("{"
                             " id_ = " "%d"
-                            " child_id = " "%d"
-                            " request_id = " "%d"
-                            " render_view_id = " "%d"
+                            " process_handle = %s"
                             " Base File = %s"
                             " }",
                             id_,
-                            process_handle_.child_id(),
-                            process_handle_.request_id(),
-                            process_handle_.render_view_id(),
+                            process_handle_.DebugString().c_str(),
                             BaseFile::DebugString().c_str());
 }