Simplify lifetime management of ValidateClientHelloResultCallback::Result objects

net/quic/core/crypto/quic_crypto_server_config.h

Index: net/quic/core/crypto/quic_crypto_server_config.h
diff --git a/net/quic/core/crypto/quic_crypto_server_config.h b/net/quic/core/crypto/quic_crypto_server_config.h
index aa820ce82c3972d089c4b10f63bd663b6146e18b..f02511a9b1f725c55da20fae0d7793905176774c 100644
--- a/net/quic/core/crypto/quic_crypto_server_config.h
+++ b/net/quic/core/crypto/quic_crypto_server_config.h
@@ -105,11 +105,11 @@ class NET_EXPORT_PRIVATE ValidateClientHelloResultCallback {
 
   ValidateClientHelloResultCallback();
   virtual ~ValidateClientHelloResultCallback();
-  void Run(const Result* result, std::unique_ptr<ProofSource::Details> details);
+  void Run(std::unique_ptr<Result> result,
+           std::unique_ptr<ProofSource::Details> details);
 
  protected:
-  virtual void RunImpl(const CryptoHandshakeMessage& client_hello,
-                       const Result& result,
+  virtual void RunImpl(std::unique_ptr<Result> result,
                        std::unique_ptr<ProofSource::Details> details) = 0;
 
  private:
@@ -518,7 +518,8 @@ class NET_EXPORT_PRIVATE QuicCryptoServerConfig {
       scoped_refptr<Config> requested_config,
       scoped_refptr<Config> primary_config,
       QuicCryptoProof* crypto_proof,
-      ValidateClientHelloResultCallback::Result* client_hello_state,
+      std::unique_ptr<ValidateClientHelloResultCallback::Result>
+          client_hello_state,
       ValidateClientHelloResultCallback* done_cb) const;
 
   // Callback class for bridging between EvaluateClientHello and
@@ -540,7 +541,8 @@ class NET_EXPORT_PRIVATE QuicCryptoServerConfig {
       QuicCryptoProof* crypto_proof,
       std::unique_ptr<ProofSource::Details> proof_source_details,
       bool get_proof_failed,
-      ValidateClientHelloResultCallback::Result* client_hello_state,
+      std::unique_ptr<ValidateClientHelloResultCallback::Result>
+          client_hello_state,
       ValidateClientHelloResultCallback* done_cb) const;
 
   // BuildRejection sets |out| to be a REJ message in reply to |client_hello|.