Certificate Transparency: Ensure timestamps over 32 bit are handled correctly.

net/cert/ct_log_response_parser_unittest.cc

Index: net/cert/ct_log_response_parser_unittest.cc
diff --git a/net/cert/ct_log_response_parser_unittest.cc b/net/cert/ct_log_response_parser_unittest.cc
index fad759019acbf9b3861aae2984cc518c7c513e5d..56dc71d6b0297e605e5a712300faf843a7d3f427 100644
--- a/net/cert/ct_log_response_parser_unittest.cc
+++ b/net/cert/ct_log_response_parser_unittest.cc
@@ -95,6 +95,23 @@ TEST(CTLogResponseParserTest, FailsToParseIncorrectLengthRootHash) {
   ASSERT_FALSE(FillSignedTreeHead(*too_short_hash_json.get(), &tree_head));
 }
 
+TEST(CTLogResponseParserTest, ParsesJsonSTHWithLargeTimestamp) {
+  SignedTreeHead tree_head;
+
+  scoped_ptr<base::Value> large_timestamp_json =
+      ParseJson(CreateSignedTreeHeadJsonString(
+          100, 1L << 34, GetSampleSTHSHA256RootHash(),

[davidben, 2016/03/02 17:30:39]

1L -> INT64_C(1) ?

[Ryan Sleevi, 2016/03/02 19:03:22]

Yes, this is needed

[Eran Messeri, 2016/03/04 11:28:30]

Done.

+          GetSampleSTHTreeHeadSignature()));
+
+  ASSERT_TRUE(FillSignedTreeHead(*large_timestamp_json.get(), &tree_head));
+
+  base::Time expected_time =
+      base::Time::UnixEpoch() +
+      base::TimeDelta::FromMilliseconds(INT64_C(1) << 34);
+
+  EXPECT_EQ(tree_head.timestamp, expected_time);
+}
+
 TEST(CTLogResponseParserTest, ParsesConsistencyProofSuccessfully) {
   std::string first(32, 'a');
   std::string second(32, 'b');