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

Unified Diff: chrome/renderer/security_filter_peer.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
« no previous file with comments | « chrome/renderer/security_filter_peer.h ('k') | content/browser/download/download_resource_handler.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/security_filter_peer.cc
diff --git a/chrome/renderer/security_filter_peer.cc b/chrome/renderer/security_filter_peer.cc
index d49b84282973c48799cf50f8605ad58eb21e3167..aef685c9d22fbaa22e47727cef3f1e0d204f1d6e 100644
--- a/chrome/renderer/security_filter_peer.cc
+++ b/chrome/renderer/security_filter_peer.cc
@@ -133,7 +133,6 @@ void BufferedPeer::OnReceivedData(std::unique_ptr<ReceivedData> data) {
void BufferedPeer::OnCompletedRequest(int error_code,
bool was_ignored_by_handler,
bool stale_copy_in_cache,
- const std::string& security_info,
const base::TimeTicks& completion_time,
int64_t total_transfer_size) {
// Give sub-classes a chance at altering the data.
@@ -141,8 +140,8 @@ void BufferedPeer::OnCompletedRequest(int error_code,
// Pretend we failed to load the resource.
original_peer_->OnReceivedResponse(response_info_);
original_peer_->OnCompletedRequest(net::ERR_ABORTED, false,
- stale_copy_in_cache, security_info,
- completion_time, total_transfer_size);
+ stale_copy_in_cache, completion_time,
+ total_transfer_size);
return;
}
@@ -152,8 +151,8 @@ void BufferedPeer::OnCompletedRequest(int error_code,
new content::FixedReceivedData(data_.data(), data_.size(), -1, 0)));
}
original_peer_->OnCompletedRequest(error_code, was_ignored_by_handler,
- stale_copy_in_cache, security_info,
- completion_time, total_transfer_size);
+ stale_copy_in_cache, completion_time,
+ total_transfer_size);
}
////////////////////////////////////////////////////////////////////////////////
@@ -181,12 +180,10 @@ void ReplaceContentPeer::OnCompletedRequest(
int error_code,
bool was_ignored_by_handler,
bool stale_copy_in_cache,
- const std::string& security_info,
const base::TimeTicks& completion_time,
int64_t total_transfer_size) {
content::ResourceResponseInfo info;
ProcessResponseInfo(info, &info, mime_type_);
- info.security_info = security_info;
info.content_length = static_cast<int>(data_.size());
original_peer_->OnReceivedResponse(info);
if (!data_.empty()) {
@@ -194,6 +191,5 @@ void ReplaceContentPeer::OnCompletedRequest(
new content::FixedReceivedData(data_.data(), data_.size(), -1, 0)));
}
original_peer_->OnCompletedRequest(net::OK, false, stale_copy_in_cache,
- security_info, completion_time,
- total_transfer_size);
+ completion_time, total_transfer_size);
}
« no previous file with comments | « chrome/renderer/security_filter_peer.h ('k') | content/browser/download/download_resource_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698