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

Unified Diff: net/ocsp/nss_ocsp.cc

Issue 398903002: Plumb redirect info out of net, through content, and into child processes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add a TODO Created 6 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
Index: net/ocsp/nss_ocsp.cc
diff --git a/net/ocsp/nss_ocsp.cc b/net/ocsp/nss_ocsp.cc
index 33d1933f348ebbae152f5f5be3404e5f7e82958d..536e0621987267897fe9c28a0969d880d7f0d147 100644
--- a/net/ocsp/nss_ocsp.cc
+++ b/net/ocsp/nss_ocsp.cc
@@ -37,6 +37,7 @@
#include "net/base/upload_data_stream.h"
#include "net/http/http_request_headers.h"
#include "net/http/http_response_headers.h"
+#include "net/url_request/redirect_info.h"
#include "net/url_request/url_request.h"
#include "net/url_request/url_request_context.h"
#include "url/gurl.h"
@@ -283,12 +284,12 @@ class OCSPRequestSession
}
virtual void OnReceivedRedirect(URLRequest* request,
- const GURL& new_url,
+ const net::RedirectInfo& redirect_info,
mmenke 2014/07/17 16:55:09 "net::" doesn't look to be needed.
davidben 2014/07/17 22:03:55 Done.
bool* defer_redirect) OVERRIDE {
DCHECK_EQ(request, request_);
DCHECK_EQ(base::MessageLoopForIO::current(), io_loop_);
- if (!new_url.SchemeIs("http")) {
+ if (!redirect_info.url.SchemeIs("http")) {
// Prevent redirects to non-HTTP schemes, including HTTPS. This matches
// the initial check in OCSPServerSession::CreateRequest().
CancelURLRequest();

Powered by Google App Engine
This is Rietveld 408576698