Use different source types for each ConnectJob subclass.

net/cert/multi_log_ct_verifier_unittest.cc

Index: net/cert/multi_log_ct_verifier_unittest.cc
diff --git a/net/cert/multi_log_ct_verifier_unittest.cc b/net/cert/multi_log_ct_verifier_unittest.cc
index 2efd7012a46155c25617a32e3ab6f2357d627e89..b0ae680e02d32c52a4d4f8a586ba801b84f13cea 100644
--- a/net/cert/multi_log_ct_verifier_unittest.cc
+++ b/net/cert/multi_log_ct_verifier_unittest.cc
@@ -122,8 +122,8 @@ class MultiLogCTVerifierTest : public ::testing::Test {
   bool VerifySinglePrecertificateChain(scoped_refptr<X509Certificate> chain) {
     SignedCertificateTimestampAndStatusList scts;
     TestNetLog test_net_log;
-    NetLogWithSource net_log =
-        NetLogWithSource::Make(&test_net_log, NetLogSourceType::CONNECT_JOB);
+    NetLogWithSource net_log = NetLogWithSource::Make(
+        &test_net_log, NetLogSourceType::SSL_CONNECT_JOB);

[mmenke, 2016/11/30 15:09:24]

Do these actually matter, or could you just use NetLotWithSource() inline,
below?

[davidben, 2016/11/30 17:37:49]

This one doesn't seem to, but the one below checks test_net_log.

 
     return verifier_->Verify(chain.get(), std::string(), std::string(), &scts,
                              net_log) == OK;
@@ -132,8 +132,8 @@ class MultiLogCTVerifierTest : public ::testing::Test {
   bool CheckPrecertificateVerification(scoped_refptr<X509Certificate> chain) {
     SignedCertificateTimestampAndStatusList scts;
     TestNetLog test_net_log;
-    NetLogWithSource net_log =
-        NetLogWithSource::Make(&test_net_log, NetLogSourceType::CONNECT_JOB);
+    NetLogWithSource net_log = NetLogWithSource::Make(
+        &test_net_log, NetLogSourceType::SSL_CONNECT_JOB);
     return (VerifySinglePrecertificateChain(chain, net_log, &scts) &&
             ct::CheckForSingleVerifiedSCTInResult(scts, kLogDescription) &&
             ct::CheckForSCTOrigin(