Disable storing cookies from URLFetchers that run with a profile's cookie jar.

chrome/browser/autofill/autofill_download.cc

Index: chrome/browser/autofill/autofill_download.cc
diff --git a/chrome/browser/autofill/autofill_download.cc b/chrome/browser/autofill/autofill_download.cc
index 14ff9a03baebf6b67e041cfc420318c72373825f..16d33a632803c4a7d7576dffae4497dadac42f6e 100644
--- a/chrome/browser/autofill/autofill_download.cc
+++ b/chrome/browser/autofill/autofill_download.cc
@@ -20,6 +20,7 @@
 #include "chrome/common/pref_names.h"
 #include "content/public/common/url_fetcher.h"
 #include "googleurl/src/gurl.h"
+#include "net/base/load_flags.h"
 #include "net/http/http_response_headers.h"
 #include "third_party/libjingle/source/talk/xmllite/xmlparser.h"
 
@@ -177,6 +178,7 @@ bool AutofillDownloadManager::StartRequest(
   fetcher->SetAutomaticallyRetryOn5xx(false);
   fetcher->SetRequestContext(request_context);
   fetcher->SetUploadData("text/plain", form_xml);
+  fetcher->SetLoadFlags(net::LOAD_DO_NOT_SAVE_COOKIES);

[Ilya Sherman, 2012/02/29 22:45:27]

Do we want net::LOAD_DO_NOT_SEND_COOKIES as well?

[jochen (gone - plz use gerrit), 2012/02/29 22:52:26]

I don't know this code enough. I'd like to merge this into M17, and I'd rather
brea as little as possible

[Ilya Sherman, 2012/02/29 22:58:28]

I'm fairly sure not sending cookies should be safe for Autofill at least.  On
the other hand, if you're planning to merge this to M17, I agree that it might
be safer to disable sending cookies in a separate CL that doesn't get merged.

[GeorgeY, 2012/02/29 23:09:52]

Yes, Autofill does not use cookies at all, so it is safe to add
net::LOAD_DO_NOT_SEND_COOKIES, though I agree that it should be done in separate
cl.

   fetcher->Start();
   return true;
 }