Cancel prerender when we discover a download starting from a page we are prer...

chrome/browser/renderer_host/download_throttling_resource_handler.h

Index: chrome/browser/renderer_host/download_throttling_resource_handler.h
===================================================================
--- chrome/browser/renderer_host/download_throttling_resource_handler.h	(revision 74194)
+++ chrome/browser/renderer_host/download_throttling_resource_handler.h	(working copy)
@@ -10,10 +10,12 @@
 
 #include "chrome/browser/download/download_request_limiter.h"
 #include "chrome/browser/renderer_host/resource_handler.h"
+#include "chrome/common/notification_type.h"
 #include "googleurl/src/gurl.h"
 
 class DownloadResourceHandler;
 class ResourceDispatcherHost;
+class ResourceRequestDetails;
 
 namespace net {
 class URLRequest;
@@ -65,6 +67,12 @@
 
   void CopyTmpBufferToDownloadHandler();
 
+  ResourceRequestDetails* GetResourceRequestDetails();
+
+  template<NotificationType::Type T>

[cbentzel, 2011/02/09 12:11:14]

Move this into .cc file and put in anonymous namespace. It's only being used by
internal methods.

It also doesn't look like there are substantial gains templating it on
NotificationType::Type - why not pass it in as an argument?

[dominich, 2011/02/09 22:33:30]

I was following the pattern established in the ResourceDispatcherHost to be
consistent, but yes I'd rather this was a parameter too. 

[dominich, 2011/02/09 22:33:30]

Done.

+  static void NotifyOnUI(int process_id, int view_id,
+                         ResourceRequestDetails* details);
+
   ResourceDispatcherHost* host_;
   net::URLRequest* request_;
   GURL url_;