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

Unified Diff: content/browser/loader/cross_site_resource_handler.cc

Issue 2315443003: Stop sending serialized SSLStatus to the renderer. (Closed)
Patch Set: self review fix and merge fix Created 4 years, 3 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: content/browser/loader/cross_site_resource_handler.cc
diff --git a/content/browser/loader/cross_site_resource_handler.cc b/content/browser/loader/cross_site_resource_handler.cc
index 67d364a0ec0c40fabbbaf2e63e7c8cc1f9033eb1..ecffa2095fcb199ad1d9fc2b639fb64f6677fa39 100644
--- a/content/browser/loader/cross_site_resource_handler.cc
+++ b/content/browser/loader/cross_site_resource_handler.cc
@@ -239,18 +239,16 @@ bool CrossSiteResourceHandler::OnReadCompleted(int bytes_read, bool* defer) {
void CrossSiteResourceHandler::OnResponseCompleted(
const net::URLRequestStatus& status,
- const std::string& security_info,
bool* defer) {
if (!in_cross_site_transition_) {
// If we're not transferring, then we should pass this through.
- next_handler_->OnResponseCompleted(status, security_info, defer);
+ next_handler_->OnResponseCompleted(status, defer);
return;
}
// We have to buffer the call until after the transition completes.
completed_during_transition_ = true;
completed_status_ = status;
- completed_security_info_ = security_info;
// Defer to tell RDH not to notify the world or clean up the pending request.
// We will do so in ResumeResponse.
@@ -288,7 +286,6 @@ void CrossSiteResourceHandler::ResumeResponse() {
if (completed_during_transition_) {
bool defer = false;
next_handler_->OnResponseCompleted(completed_status_,
- completed_security_info_,
&defer);
if (!defer)
ResumeIfDeferred();
« no previous file with comments | « content/browser/loader/cross_site_resource_handler.h ('k') | content/browser/loader/detachable_resource_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698