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

Unified Diff: content/browser/loader/detachable_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/detachable_resource_handler.cc
diff --git a/content/browser/loader/detachable_resource_handler.cc b/content/browser/loader/detachable_resource_handler.cc
index 82544549b08c703e44f50e2ff6d334fc2485abcb..2e1af5e7c6c396821e762203c1d9628012f1093b 100644
--- a/content/browser/loader/detachable_resource_handler.cc
+++ b/content/browser/loader/detachable_resource_handler.cc
@@ -47,7 +47,7 @@ void DetachableResourceHandler::Detach() {
net::URLRequestStatus status(net::URLRequestStatus::CANCELED,
net::ERR_ABORTED);
bool defer_ignored = false;
- next_handler_->OnResponseCompleted(status, std::string(), &defer_ignored);
+ next_handler_->OnResponseCompleted(status, &defer_ignored);
DCHECK(!defer_ignored);
// If |next_handler_| were to defer its shutdown in OnResponseCompleted,
// this would destroy it anyway. Fortunately, AsyncResourceHandler never
@@ -157,7 +157,6 @@ bool DetachableResourceHandler::OnReadCompleted(int bytes_read, bool* defer) {
void DetachableResourceHandler::OnResponseCompleted(
const net::URLRequestStatus& status,
- const std::string& security_info,
bool* defer) {
// No DCHECK(!is_deferred_) as the request may have been cancelled while
// deferred.
@@ -167,7 +166,7 @@ void DetachableResourceHandler::OnResponseCompleted(
is_finished_ = true;
- next_handler_->OnResponseCompleted(status, security_info, &is_deferred_);
+ next_handler_->OnResponseCompleted(status, &is_deferred_);
*defer = is_deferred_;
}
« no previous file with comments | « content/browser/loader/detachable_resource_handler.h ('k') | content/browser/loader/intercepting_resource_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698