Do cookie checks in NetworkDelegate instead of the URLRequest::Delegate.

chrome/browser/search_engines/template_url_fetcher.h

Index: chrome/browser/search_engines/template_url_fetcher.h
diff --git a/chrome/browser/search_engines/template_url_fetcher.h b/chrome/browser/search_engines/template_url_fetcher.h
index d82ef0a5686eb0be8f109f7844fc04b66ebd146e..14e6371542b6b83ce5a802f5f5a82ed3cf41867b 100644
--- a/chrome/browser/search_engines/template_url_fetcher.h
+++ b/chrome/browser/search_engines/template_url_fetcher.h
@@ -15,6 +15,10 @@ class Profile;
 class TemplateURL;
 class TemplateURLFetcherCallbacks;
 
+namespace content {
+class WebContents;
+}
+
 // TemplateURLFetcher is responsible for downloading OpenSearch description
 // documents, creating a TemplateURL from the OSDD, and adding the TemplateURL
 // to the TemplateURLService. Downloading is done in the background.
@@ -36,6 +40,7 @@ class TemplateURLFetcher {
   void ScheduleDownload(const string16& keyword,
                         const GURL& osdd_url,
                         const GURL& favicon_url,
+                        content::WebContents* web_contents,

[Peter Kasting, 2012/03/08 20:46:27]

Nit: Can you add a comment about what this argument means?  Also your
implementation comments that this can be NULL during tests, which implies that
it shouldn't be NULL in normal use; you might want to comment about that as
well.

                         TemplateURLFetcherCallbacks* callbacks,
                         ProviderType provider_type);