Optimize CT & OCSP handling code

net/cert/do_nothing_ct_verifier.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_CERT_DO_NOTHING_CT_VERIFIER_H_
 #define NET_CERT_DO_NOTHING_CT_VERIFIER_H_
 
 #include "base/macros.h"
 #include "net/base/net_export.h"
 #include "net/cert/ct_verifier.h"
@@ skipping 32 matching lines @@
 // expected that classes which expect a CTVerifier will require one to be
 // supplied, forcing the caller to make an intentional and explicit decision
 // about the appropriate security policy, rather than leaving it ambiguous,
 // such as via a nullptr. This class is intended to indicate an intentional
 // consideration of CT, and a decision to not support it.
 class NET_EXPORT DoNothingCTVerifier : public CTVerifier {
  public:
   DoNothingCTVerifier();
   ~DoNothingCTVerifier() override;
 
-  int Verify(X509Certificate* cert,
-             const std::string& stapled_ocsp_response,
-             const std::string& sct_list_from_tls_extension,
-             SignedCertificateTimestampAndStatusList* output_scts,
-             const NetLogWithSource& net_log) override;
+  void Verify(X509Certificate* cert,
+              base::StringPiece stapled_ocsp_response,
+              base::StringPiece sct_list_from_tls_extension,
+              SignedCertificateTimestampAndStatusList* output_scts,
+              const NetLogWithSource& net_log) override;
+
+  void SetObserver(Observer* observer) override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(DoNothingCTVerifier);
 };
 
 }  // namespace net
 
 #endif  // NET_CERT_DO_NOTHING_CT_VERIFIER_H_