Minor cleanups to variation_service.cc.

chrome/browser/metrics/variations_service.cc

Index: chrome/browser/metrics/variations_service.cc
===================================================================
--- chrome/browser/metrics/variations_service.cc	(revision 135898)
+++ chrome/browser/metrics/variations_service.cc	(working copy)
@@ -44,6 +44,11 @@
   return chrome::VersionInfo::CHANNEL_UNKNOWN;
 }
 
+// Converts |date_time| in chrome_variations::Study date format to base::Time.
+base::Time ConvertStudyDateToBaseTime(int64 date_time) {
+  return base::Time::UnixEpoch() + base::TimeDelta::FromSeconds(date_time);
+}
+
 }  // namespace
 
 // Static
@@ -162,7 +167,7 @@
                                           const std::string& version_string) {
   const Version current_version(version_string);
   if (!current_version.IsValid()) {
-    DCHECK(false);
+    NOTREACHED();
     return false;
   }
 
@@ -188,18 +193,16 @@
 // static
 bool VariationsService::CheckStudyDate(const chrome_variations::Study& study,
                                        const base::Time& date_time) {
-  const base::Time epoch = base::Time::UnixEpoch();
-
   if (study.has_start_date()) {
     const base::Time start_date =
-        epoch + base::TimeDelta::FromSeconds(study.start_date());
+        ConvertStudyDateToBaseTime(study.start_date());
     if (date_time < start_date)
       return false;
   }
 
   if (study.has_expiry_date()) {
     const base::Time expiry_date =
-        epoch + base::TimeDelta::FromSeconds(study.expiry_date());
+        ConvertStudyDateToBaseTime(study.expiry_date());
     if (date_time >= expiry_date)
       return false;
   }
@@ -253,9 +256,8 @@
     return;
   }
 
-  const base::Time epoch = base::Time::UnixEpoch();
   const base::Time expiry_date =
-      epoch + base::TimeDelta::FromSeconds(study.expiry_date());
+      ConvertStudyDateToBaseTime(study.expiry_date());
   base::Time::Exploded exploded_end_date;
   expiry_date.UTCExplode(&exploded_end_date);