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

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

Issue 25536005: Clean up ResourceHandler API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 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
Index: content/browser/loader/certificate_resource_handler.cc
diff --git a/content/browser/loader/certificate_resource_handler.cc b/content/browser/loader/certificate_resource_handler.cc
index d850b19a82ee5b3be8e2deef613dd26790664876..1688d72eae4bf5ff0ec1db43afd36cdb569187a6 100644
--- a/content/browser/loader/certificate_resource_handler.cc
+++ b/content/browser/loader/certificate_resource_handler.cc
@@ -19,7 +19,7 @@ namespace content {
CertificateResourceHandler::CertificateResourceHandler(
net::URLRequest* request)
- : request_(request),
+ : ResourceHandler(request),
content_length_(0),
read_buffer_(NULL),
resource_buffer_(NULL),
@@ -57,7 +57,7 @@ bool CertificateResourceHandler::OnWillStart(int request_id,
}
bool CertificateResourceHandler::OnWillRead(int request_id,
- net::IOBuffer** buf,
+ scoped_refptr<net::IOBuffer>* buf,
int* buf_size,
int min_size) {
static const int kReadBufSize = 32768;
@@ -108,9 +108,9 @@ bool CertificateResourceHandler::OnResponseCompleted(
// Note that it's up to the browser to verify that the certificate
// data is well-formed.
- const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request_);
+ const ResourceRequestInfo* info = GetRequestInfo();
GetContentClient()->browser()->AddCertificate(
- request_, cert_type_, content_bytes, content_length_,
+ request(), cert_type_, content_bytes, content_length_,
info->GetChildID(), info->GetRouteID());
return true;
« no previous file with comments | « content/browser/loader/certificate_resource_handler.h ('k') | content/browser/loader/cross_site_resource_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698