 Chromium Code Reviews
 Chromium Code Reviews Issue 1641413002:
  Makes GetBuildTime behave identically on all build types.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1641413002:
  Makes GetBuildTime behave identically on all build types.  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: base/build_time_unittest.cc | 
| diff --git a/base/build_time_unittest.cc b/base/build_time_unittest.cc | 
| index aac64a7dc7c1f63d3b664b809245680faeef43d8..a6d9dd4a65b38715a6466104ac652b4fc78596b5 100644 | 
| --- a/base/build_time_unittest.cc | 
| +++ b/base/build_time_unittest.cc | 
| @@ -7,11 +7,7 @@ | 
| #include "testing/gtest/include/gtest/gtest.h" | 
| TEST(BuildTime, DateLooksValid) { | 
| -#if !defined(DONT_EMBED_BUILD_METADATA) | 
| char build_date[] = __DATE__; | 
| 
agl
2016/01/29 21:46:46
I'm not sure when DONT_EMBED_BUILD_METADATA was de
 
M-A Ruel
2016/01/30 02:58:14
It's meant to be used on the try server.
We want
 
Zachary Forman
2016/02/01 07:28:30
This sounds pretty reasonable to me. I've implemen
 | 
| -#else | 
| - char build_date[] = "Sep 02 2008"; | 
| -#endif | 
| EXPECT_EQ(11u, strlen(build_date)); | 
| EXPECT_EQ(' ', build_date[3]); | 
| @@ -19,11 +15,7 @@ TEST(BuildTime, DateLooksValid) { | 
| } | 
| TEST(BuildTime, TimeLooksValid) { | 
| -#if defined(DONT_EMBED_BUILD_METADATA) | 
| - char build_time[] = "08:00:00"; | 
| -#else | 
| - char build_time[] = __TIME__; | 
| -#endif | 
| + char build_time[] = "00:00:00"; | 
| EXPECT_EQ(8u, strlen(build_time)); | 
| EXPECT_EQ(':', build_time[2]); |