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

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

Issue 1499793003: Fix classes that have too many virtuals for inline constructors. Base URL: https://chromium.googlesource.com/chromium/src.git@enable-virtuals-as-complexity
Patch Set: Finish fixing the codebase that's accessible from Linux. Created 5 years 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 "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/callback_helpers.h" 9 #include "base/callback_helpers.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 17 matching lines...) Expand all
28 #include "net/quic/crypto/crypto_protocol.h" 28 #include "net/quic/crypto/crypto_protocol.h"
29 #include "net/ssl/ssl_config_service.h" 29 #include "net/ssl/ssl_config_service.h"
30 30
31 using base::StringPiece; 31 using base::StringPiece;
32 using base::StringPrintf; 32 using base::StringPrintf;
33 using std::string; 33 using std::string;
34 using std::vector; 34 using std::vector;
35 35
36 namespace net { 36 namespace net {
37 37
38 ProofVerifyDetailsChromium::ProofVerifyDetailsChromium() = default;
39
38 ProofVerifyDetails* ProofVerifyDetailsChromium::Clone() const { 40 ProofVerifyDetails* ProofVerifyDetailsChromium::Clone() const {
39 ProofVerifyDetailsChromium* other = new ProofVerifyDetailsChromium; 41 ProofVerifyDetailsChromium* other = new ProofVerifyDetailsChromium;
40 other->cert_verify_result = cert_verify_result; 42 other->cert_verify_result = cert_verify_result;
41 other->ct_verify_result = ct_verify_result; 43 other->ct_verify_result = ct_verify_result;
42 return other; 44 return other;
43 } 45 }
44 46
45 // A Job handles the verification of a single proof. It is owned by the 47 // A Job handles the verification of a single proof. It is owned by the
46 // ProofVerifier. If the verification can not complete synchronously, it 48 // ProofVerifier. If the verification can not complete synchronously, it
47 // will notify the ProofVerifier upon completion. 49 // will notify the ProofVerifier upon completion.
(...skipping 374 matching lines...) Expand 10 before | Expand all | Expand 10 after
422 } 424 }
423 return status; 425 return status;
424 } 426 }
425 427
426 void ProofVerifierChromium::OnJobComplete(Job* job) { 428 void ProofVerifierChromium::OnJobComplete(Job* job) {
427 active_jobs_.erase(job); 429 active_jobs_.erase(job);
428 delete job; 430 delete job;
429 } 431 }
430 432
431 } // namespace net 433 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698