Move the sanity check on build time vs current time to the net test

net/cert/ct_policy_enforcer_unittest.cc

 // Copyright 2014 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/ct_policy_enforcer.h"
 
 #include <memory>
 #include <string>
 
-#include "base/build_time.h"
 #include "base/time/time.h"
 #include "base/version.h"
 #include "crypto/rsa_private_key.h"
 #include "crypto/sha2.h"
 #include "net/cert/ct_ev_whitelist.h"
 #include "net/cert/ct_policy_status.h"
 #include "net/cert/ct_verify_result.h"
 #include "net/cert/x509_certificate.h"
 #include "net/cert/x509_util.h"
 #include "net/log/net_log_with_source.h"
@@ skipping 32 matching lines @@
 
 const char kGoogleAviatorLogID[] =
     "\x68\xf6\x98\xf8\x1f\x64\x82\xbe\x3a\x8c\xee\xb9\x28\x1d\x4c\xfc\x71\x51"
     "\x5d\x67\x93\xd4\x44\xd1\x0a\x67\xac\xbb\x4f\x4f\xfb\xc4";
 static_assert(arraysize(kGoogleAviatorLogID) - 1 == crypto::kSHA256Length,
               "Incorrect log ID length.");
 
 class CTPolicyEnforcerTest : public ::testing::Test {
  public:
   void SetUp() override {
-    VerifyBuildIsTimely();
-
     policy_enforcer_.reset(new CTPolicyEnforcer);
 
     std::string der_test_cert(ct::GetDerEncodedX509Cert());
     chain_ = X509Certificate::CreateFromBytes(der_test_cert.data(),
                                               der_test_cert.size());
     ASSERT_TRUE(chain_.get());
     google_log_id_ = std::string(kGoogleAviatorLogID, crypto::kSHA256Length);
     non_google_log_id_.assign(crypto::kSHA256Length, 1);
   }
 
@@ skipping 52 matching lines @@
       ct::SignedCertificateTimestamp::Origin desired_origin,
       size_t num_scts,
       ct::SCTList* verified_scts) {
     std::vector<std::string> desired_log_ids;
     desired_log_ids.push_back(google_log_id_);
     FillListWithSCTsOfOrigin(desired_origin, num_scts, desired_log_ids, true,
                              verified_scts);
   }
 
  protected:
-  void VerifyBuildIsTimely() {
-    base::Time build_time = base::GetBuildTime();
-    base::Time now = base::Time::Now();
-
-    // Internally CTPolicyEnforcer expects the build time to be no older than 10
-    // weeks. If it is then many tests in this file (and other net unittests)
-    // will fail. crbug.com/666821
-    EXPECT_LT((now - build_time).InDays(), 70)
-        << "IMPORTANT: There is a problem with the system clock and/or the "
-           "build timestamp. This will lead to many net_unittests failing "
-           "(crbug.com/666821)\n"
-        << "now: " << now << ", build_time: " << build_time;
-  }
-
   std::unique_ptr<CTPolicyEnforcer> policy_enforcer_;
   scoped_refptr<X509Certificate> chain_;
   std::string google_log_id_;
   std::string non_google_log_id_;
 };
 
 #if defined(OS_ANDROID)
 #define MAYBE_DoesNotConformToCTEVPolicyNotEnoughDiverseSCTsAllGoogle \
   DISABLED_DoesNotConformToCTEVPolicyNotEnoughDiverseSCTsAllGoogle
 #else
@@ skipping 418 matching lines @@
   FillListWithSCTsOfOrigin(ct::SignedCertificateTimestamp::SCT_EMBEDDED, 2,
                            &scts);
   EXPECT_EQ(ct::EVPolicyCompliance::EV_POLICY_NOT_ENOUGH_SCTS,
             policy_enforcer_->DoesConformToCTEVPolicy(
                 chain_.get(), nullptr, scts, NetLogWithSource()));
 }
 
 }  // namespace
 
 }  // namespace net