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

Unified Diff: chrome/browser/chromeos/policy/policy_cert_verifier.cc

Issue 1115903002: Refactor the API for CertVerifier::Verify() and the implementation of MultiThreadedCertVerifier::Ver (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase again Created 5 years, 7 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: chrome/browser/chromeos/policy/policy_cert_verifier.cc
diff --git a/chrome/browser/chromeos/policy/policy_cert_verifier.cc b/chrome/browser/chromeos/policy/policy_cert_verifier.cc
index d3abda8e4c21d532eb5e7b1c54086b9742eeb00c..3165fa9ea4ffae41035807d7225bad5c25d24692 100644
--- a/chrome/browser/chromeos/policy/policy_cert_verifier.cc
+++ b/chrome/browser/chromeos/policy/policy_cert_verifier.cc
@@ -76,7 +76,7 @@ int PolicyCertVerifier::Verify(
net::CRLSet* crl_set,
net::CertVerifyResult* verify_result,
const net::CompletionCallback& completion_callback,
- RequestHandle* out_req,
+ scoped_ptr<Request>* out_req,
const net::BoundNetLog& net_log) {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
DCHECK(delegate_);
@@ -92,11 +92,6 @@ int PolicyCertVerifier::Verify(
return error;
}
-void PolicyCertVerifier::CancelRequest(RequestHandle req) {
- DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
- delegate_->CancelRequest(req);
-}
-
bool PolicyCertVerifier::SupportsOCSPStapling() {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::IO));
return delegate_->SupportsOCSPStapling();

Powered by Google App Engine
This is Rietveld 408576698