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

Unified Diff: chrome/browser/automation/url_request_automation_job.cc

Issue 10830124: Removed checks that verify URLRequest has a Context (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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
« no previous file with comments | « no previous file | net/url_request/url_request.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/automation/url_request_automation_job.cc
diff --git a/chrome/browser/automation/url_request_automation_job.cc b/chrome/browser/automation/url_request_automation_job.cc
index f5856bd2409f3643327f9fa12f198e53514c17a0..8b41e57c54151dc9dada0ffb5736f99c6e652651 100644
--- a/chrome/browser/automation/url_request_automation_job.cc
+++ b/chrome/browser/automation/url_request_automation_job.cc
@@ -416,21 +416,19 @@ void URLRequestAutomationJob::StartAsync() {
for (size_t i = 0; i < arraysize(kFilteredHeaderStrings); ++i)
new_request_headers.RemoveHeader(kFilteredHeaderStrings[i]);
- if (request_->context()) {
- // Only add default Accept-Language and Accept-Charset if the request
- // didn't have them specified.
- if (!new_request_headers.HasHeader(
- net::HttpRequestHeaders::kAcceptLanguage) &&
- !request_->context()->accept_language().empty()) {
- new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptLanguage,
- request_->context()->accept_language());
- }
- if (!new_request_headers.HasHeader(
- net::HttpRequestHeaders::kAcceptCharset) &&
- !request_->context()->accept_charset().empty()) {
- new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptCharset,
- request_->context()->accept_charset());
- }
+ // Only add default Accept-Language and Accept-Charset if the request
+ // didn't have them specified.
+ if (!new_request_headers.HasHeader(
+ net::HttpRequestHeaders::kAcceptLanguage) &&
+ !request_->context()->accept_language().empty()) {
+ new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptLanguage,
+ request_->context()->accept_language());
+ }
+ if (!new_request_headers.HasHeader(
+ net::HttpRequestHeaders::kAcceptCharset) &&
+ !request_->context()->accept_charset().empty()) {
+ new_request_headers.SetHeader(net::HttpRequestHeaders::kAcceptCharset,
+ request_->context()->accept_charset());
}
// Ensure that we do not send username and password fields in the referrer.
« no previous file with comments | « no previous file | net/url_request/url_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698