Update CertVerifier::Verify to use RequestParams instead

net/cert/cert_verifier_unittest.cc

 // 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.
 
 #include "net/cert/cert_verifier.h"
 
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "net/base/test_data_directory.h"
 #include "net/cert/x509_certificate.h"
@@ skipping 33 matching lines @@
     CertVerifier::RequestParams key2;
 
     // Expectation:
     // -1 means key1 is less than key2
     //  0 means key1 equals key2
     //  1 means key1 is greater than key2
     int expected_result;
   } tests[] = {
       {
           // Test for basic equivalence.
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
           0,
       },
       {
           // Test that different certificates but with the same CA and for
           // the same host are different validation keys.
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
-          CertVerifier::RequestParams(expired_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(expired_cert, "www.example.test", 0,
                                       std::string(), empty_list),
           -1,
       },
       {
           // Test that the same EE certificate for the same host, but with
           // different chains are different validation keys.
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
-          CertVerifier::RequestParams(combined_cert.get(), "www.example.test",
-                                      0, std::string(), empty_list),
+          CertVerifier::RequestParams(combined_cert, "www.example.test", 0,
+                                      std::string(), empty_list),
           1,
       },
       {
           // The same certificate, with the same chain, but for different
           // hosts are different validation keys.
-          CertVerifier::RequestParams(ok_cert.get(), "www1.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www1.example.test", 0,
                                       std::string(), empty_list),
-          CertVerifier::RequestParams(ok_cert.get(), "www2.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www2.example.test", 0,
                                       std::string(), empty_list),
           -1,
       },
       {
           // The same certificate, chain, and host, but with different flags
           // are different validation keys.
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test",
+          CertVerifier::RequestParams(ok_cert, "www.example.test",
                                       CertVerifier::VERIFY_EV_CERT,
                                       std::string(), empty_list),
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
           1,
       },
       {
           // Different additional_trust_anchors.
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), test_list),
           -1,
       },
       {
           // Different OCSP responses.
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       "ocsp response", empty_list),
-          CertVerifier::RequestParams(ok_cert.get(), "www.example.test", 0,
+          CertVerifier::RequestParams(ok_cert, "www.example.test", 0,
                                       std::string(), empty_list),
           -1,
       },
   };
   for (size_t i = 0; i < arraysize(tests); ++i) {
     SCOPED_TRACE(i);
 
     const CertVerifier::RequestParams& key1 = tests[i].key1;
     const CertVerifier::RequestParams& key2 = tests[i].key2;
 
@@ skipping 13 matching lines @@
         EXPECT_FALSE(key1 < key2);
         EXPECT_TRUE(key2 < key1);
         break;
       default:
         FAIL() << "Invalid expectation. Can be only -1, 0, 1";
     }
   }
 }
 
 }  // namespace net