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

Unified Diff: webrtc/base/httprequest.cc

Issue 1396683005: Get rid of deprecated HttpClient fail_redirect accessors. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: ALWAYS Created 5 years, 2 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 | « webrtc/base/httpclient.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/httprequest.cc
diff --git a/webrtc/base/httprequest.cc b/webrtc/base/httprequest.cc
index 3199c8f94f5b403c1774f489242c1db059db50d7..0139f01fcf8d8072b717a48f25b312e9b6cc490e 100644
--- a/webrtc/base/httprequest.cc
+++ b/webrtc/base/httprequest.cc
@@ -48,11 +48,13 @@ void HttpMonitor::OnHttpClientComplete(HttpClient * http, HttpErrorType error) {
const int kDefaultHTTPTimeout = 30 * 1000; // 30 sec
-HttpRequest::HttpRequest(const std::string &user_agent)
- : firewall_(0), port_(80), secure_(false),
- timeout_(kDefaultHTTPTimeout), fail_redirect_(false),
- client_(user_agent.c_str(), NULL), error_(HE_NONE) {
-}
+HttpRequest::HttpRequest(const std::string& user_agent)
+ : firewall_(0),
+ port_(80),
+ secure_(false),
+ timeout_(kDefaultHTTPTimeout),
+ client_(user_agent.c_str(), NULL),
+ error_(HE_NONE) {}
HttpRequest::~HttpRequest() = default;
@@ -82,7 +84,7 @@ void HttpRequest::Send() {
if (transparent_proxy) {
client_.set_proxy(proxy_);
}
- client_.set_fail_redirect(fail_redirect_);
+ client_.set_redirect_action(HttpClient::REDIRECT_ALWAYS);
SocketAddress server(host_, port_);
client_.set_server(server);
« no previous file with comments | « webrtc/base/httpclient.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698