Commit 36ec7c8b authored by Matt Mueller's avatar Matt Mueller Committed by Commit Bot

Add _usec suffix to trial_verification_time field in cert_logger.proto

Bug: 1008994
Change-Id: I10a52af910605852786ffcf888276764d6660c39
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835158Reviewed-by: default avatarEmily Stark <estark@chromium.org>
Commit-Queue: Matt Mueller <mattm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#702208}
parent d99fdbae
......@@ -234,7 +234,7 @@ message TrialVerificationInfo {
// The time (in usec since the Windows epoch) when the trial verifier
// attempted to verify the chain.
optional int64 trial_verification_time = 8;
optional int64 trial_verification_time_usec = 8;
// The GeneralizedTime encoded time when the trial verifier attempted to
// verify the chain.
......
......@@ -194,7 +194,7 @@ CertificateErrorReport::CertificateErrorReport(
trial_report->mutable_mac_combined_trust_debug_info());
#endif
if (!debug_info->trial_verification_time.is_null()) {
trial_report->set_trial_verification_time(
trial_report->set_trial_verification_time_usec(
debug_info->trial_verification_time.ToDeltaSinceWindowsEpoch()
.InMicroseconds());
}
......
......@@ -379,9 +379,9 @@ TEST(ErrorReportTest, TrialDebugInfo) {
#else
EXPECT_EQ(0, trial_info.mac_combined_trust_debug_info_size());
#endif
ASSERT_TRUE(trial_info.has_trial_verification_time());
ASSERT_TRUE(trial_info.has_trial_verification_time_usec());
EXPECT_EQ(time.ToDeltaSinceWindowsEpoch().InMicroseconds(),
trial_info.trial_verification_time());
trial_info.trial_verification_time_usec());
ASSERT_TRUE(trial_info.has_trial_der_verification_time());
EXPECT_EQ("it's just a string", trial_info.trial_der_verification_time());
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment