Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(127)

Unified Diff: net/url_request/report_sender.cc

Issue 2851493003: Remove unused CookiePreferences from report sender and never send cookies (Closed)
Patch Set: Fix iOS build Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/url_request/report_sender.cc
diff --git a/net/url_request/report_sender.cc b/net/url_request/report_sender.cc
index 96263d7f2b4ea4b1f87e98508e4723613cd3b0dc..bc2bbc35a438d17697eae0e820800fcdbb826f58 100644
--- a/net/url_request/report_sender.cc
+++ b/net/url_request/report_sender.cc
@@ -39,10 +39,8 @@ class CallbackInfo : public base::SupportsUserData::Data {
namespace net {
-ReportSender::ReportSender(URLRequestContext* request_context,
- CookiesPreference cookies_preference)
- : request_context_(request_context),
- cookies_preference_(cookies_preference) {}
+ReportSender::ReportSender(URLRequestContext* request_context)
+ : request_context_(request_context) {}
ReportSender::~ReportSender() {
}
@@ -58,12 +56,9 @@ void ReportSender::Send(const GURL& report_uri,
url_request->SetUserData(&kUserDataKey,
new CallbackInfo(success_callback, error_callback));
- int load_flags =
- LOAD_BYPASS_CACHE | LOAD_DISABLE_CACHE | LOAD_DO_NOT_SEND_AUTH_DATA;
- if (cookies_preference_ != SEND_COOKIES) {
- load_flags |= LOAD_DO_NOT_SEND_COOKIES | LOAD_DO_NOT_SAVE_COOKIES;
- }
- url_request->SetLoadFlags(load_flags);
+ url_request->SetLoadFlags(
+ LOAD_BYPASS_CACHE | LOAD_DISABLE_CACHE | LOAD_DO_NOT_SEND_AUTH_DATA |
+ LOAD_DO_NOT_SEND_COOKIES | LOAD_DO_NOT_SAVE_COOKIES);
HttpRequestHeaders extra_headers;
extra_headers.SetHeader(HttpRequestHeaders::kContentType, content_type);

Powered by Google App Engine
This is Rietveld 408576698