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

Side by Side Diff: net/quic/crypto/proof_verifier_chromium.cc

Issue 1994353002: Update CertVerifier::Verify to use RequestParams instead (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@request_params
Patch Set: Rebased Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/quic/crypto/proof_verifier_chromium.h" 5 #include "net/quic/crypto/proof_verifier_chromium.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 281 matching lines...) Expand 10 before | Expand all | Expand 10 after
292 callback->Run(rv == OK, error_details_, &verify_details); 292 callback->Run(rv == OK, error_details_, &verify_details);
293 // Will delete |this|. 293 // Will delete |this|.
294 proof_verifier_->OnJobComplete(this); 294 proof_verifier_->OnJobComplete(this);
295 } 295 }
296 } 296 }
297 297
298 int ProofVerifierChromium::Job::DoVerifyCert(int result) { 298 int ProofVerifierChromium::Job::DoVerifyCert(int result) {
299 next_state_ = STATE_VERIFY_CERT_COMPLETE; 299 next_state_ = STATE_VERIFY_CERT_COMPLETE;
300 300
301 return verifier_->Verify( 301 return verifier_->Verify(
302 cert_.get(), hostname_, std::string(), cert_verify_flags_, 302 net::CertVerifier::RequestParams(cert_, hostname_, cert_verify_flags_,
303 std::string(), CertificateList()),
303 SSLConfigService::GetCRLSet().get(), &verify_details_->cert_verify_result, 304 SSLConfigService::GetCRLSet().get(), &verify_details_->cert_verify_result,
304 base::Bind(&ProofVerifierChromium::Job::OnIOComplete, 305 base::Bind(&ProofVerifierChromium::Job::OnIOComplete,
305 base::Unretained(this)), 306 base::Unretained(this)),
306 &cert_verifier_request_, net_log_); 307 &cert_verifier_request_, net_log_);
307 } 308 }
308 309
309 int ProofVerifierChromium::Job::DoVerifyCertComplete(int result) { 310 int ProofVerifierChromium::Job::DoVerifyCertComplete(int result) {
310 cert_verifier_request_.reset(); 311 cert_verifier_request_.reset();
311 312
312 const CertVerifyResult& cert_verify_result = 313 const CertVerifyResult& cert_verify_result =
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
483 } 484 }
484 return status; 485 return status;
485 } 486 }
486 487
487 void ProofVerifierChromium::OnJobComplete(Job* job) { 488 void ProofVerifierChromium::OnJobComplete(Job* job) {
488 active_jobs_.erase(job); 489 active_jobs_.erase(job);
489 delete job; 490 delete job;
490 } 491 }
491 492
492 } // namespace net 493 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698