Add downloads.onCreated apitest and fix a few bugs

content/browser/download/download_request_handle.cc

Index: content/browser/download/download_request_handle.cc
diff --git a/content/browser/download/download_request_handle.cc b/content/browser/download/download_request_handle.cc
index e5d9b60d1cac3937d3252300518fe33f7ec4bf80..3f5d93aae7aecce2c7674d38e94c671d62e322f1 100644
--- a/content/browser/download/download_request_handle.cc
+++ b/content/browser/download/download_request_handle.cc
@@ -61,15 +61,8 @@ TabContents* DownloadRequestHandle::GetTabContents() const {
 }
 
 DownloadManager* DownloadRequestHandle::GetDownloadManager() const {
-  TabContents* contents = GetTabContents();
-  if (!contents)
-    return NULL;
-
-  content::BrowserContext* browser_context = contents->browser_context();
-  if (!browser_context)
-    return NULL;
-
-  return browser_context->GetDownloadManager();
+  return RenderViewHost::FromID(child_id_, render_view_id_)->process()
+    ->browser_context()->GetDownloadManager();

[Randy Smith (Not in Mondays), 2011/10/17 17:39:56]

Random thought that you're welcome to bounce as not being appropriate for this
CL: Now that we have global ids that include the DownloadManager, could we nuke
this method on DownloadRequestHandle and just get the DM from there?  I *think*
we have access to a global_id everywhere we have a DownloadRequestHandle.  

[benjhayden, 2011/10/17 17:54:29]

I thought that we decided that that would be breaking the DownloadId
abstraction? Meh, we can do it in DownloadId part 2 if that SGTY. I'd like to
try to keep this cl small.

 }
 
 void DownloadRequestHandle::PauseRequest() const {