Revert of QUIC - minor debugging logic to see why SCT verification is failing when (patchset #2 id:20001 of https://codereview.chromium.org/1949333002/ )
Reason for revert:
Reverting debugging logic.
Original issue's description:
> QUIC - minor debugging logic to see why SCT verification is failing when
> QUIC is accessed from home.
>
> Will delete this code asap.
>
> [email protected]
>
> Committed: https://crrev.com/3903514d7dda980ff099152f69bdfb4964c6c707
> Cr-Commit-Position: refs/heads/master@{#391950}
[email protected]
# Not skipping CQ checks because original CL landed more than 1 days ago.
Review-Url: https://codereview.chromium.org/1999573002
Cr-Commit-Position: refs/heads/master@{#394975}
diff --git a/net/quic/crypto/proof_verifier_chromium.cc b/net/quic/crypto/proof_verifier_chromium.cc
index e6e9516e..93d9721 100644
--- a/net/quic/crypto/proof_verifier_chromium.cc
+++ b/net/quic/crypto/proof_verifier_chromium.cc
@@ -212,22 +212,9 @@
// Note that this is a completely synchronous operation: The CT Log Verifier
// gets all the data it needs for SCT verification and does not do any
// external communication.
- int result = cert_transparency_verifier_->Verify(
- cert_.get(), std::string(), cert_sct,
- &verify_details_->ct_verify_result, net_log_);
- // TODO(rtenneti): Delete this debugging code.
- if (result == OK) {
- VLOG(1) << "CTVerifier::Verify success";
- } else {
- VLOG(1) << "CTVerifier::Verify failed: " << result;
- }
- } else {
- // TODO(rtenneti): Delete this debugging code.
- if (cert_transparency_verifier_) {
- VLOG(1) << "cert_sct is empty";
- } else {
- VLOG(1) << "cert_transparency_verifier_ is null";
- }
+ cert_transparency_verifier_->Verify(cert_.get(), std::string(), cert_sct,
+ &verify_details_->ct_verify_result,
+ net_log_);
}
// We call VerifySignature first to avoid copying of server_config and
