Remove old daylightSavingTA() (facebook#1245)

Summary: Pull Request resolved: facebook#1245

Differential Revision: D52579225

include/hermes/VM/JSLib/DateUtil.h

@@ -112,10 +112,6 @@ double localTZA();
 //===----------------------------------------------------------------------===//
 // ES5.1 15.9.1.8
 
-/// Daylight saving time adjustment, in milliseconds, at time \p t.
-/// \param t timestamp in milliseconds.
-double daylightSavingTA(double t);
-
 namespace detail {
 // Exposed for test only
 int32_t equivalentTime(int64_t epochSecs);

lib/VM/JSLib/DateUtil.cpp

@@ -372,50 +372,6 @@ int32_t detail::equivalentTime(int64_t epochSecs) {
   return (eqYearAsEpochDays + dayOfYear) * SECONDS_PER_DAY + secsOfDay;
 }
 
-double daylightSavingTA(double t) {
-  // The spec says LocalTime should only take finite time value and return 0 in
-  // case conversion fails. Once we enforce the finite input at the caller site,
-  // we should remove the below check or replace it with an assertion. For now,
-  // let's return NaN instead if it's not finite value.
-  if (!std::isfinite(t)) {
-    return std::numeric_limits<double>::quiet_NaN();
-  }
-
-  // Convert t to seconds and get the actual time needed.
-  const double seconds = t / MS_PER_SECOND;
-  // If the number of seconds is higher or lower than a unix timestamp can
-  // support, clamp it. This is not correct in all cases, but returning NaN (for
-  // Invalid Date) breaks date construction entirely. Clamping only results in
-  // small errors in daylight savings time. This is only a problem in systems
-  // with a 32-bit time_t, like some Android systems.
-  time_t local = 0;
-  if (seconds > TIME_RANGE_SECS || seconds < -TIME_RANGE_SECS) {
-    // Return NaN if input is outside Time Range allowed in ES5.1
-    return std::numeric_limits<double>::quiet_NaN();
-  }
-  // This will truncate any fractional seconds, which is ok for daylight
-  // savings time calculations.
-  local = detail::equivalentTime(static_cast<int64_t>(seconds));
-
-  std::tm tm;
-#ifdef _WINDOWS
-  // The return value of localtime_s on Windows is an error code instead of
-  // a pointer to std::tm. For simplicity, we don't inspect the concrete error
-  // code and just return 0.
-  auto err = ::localtime_s(&tm, &local);
-  if (err) {
-    return 0;
-  }
-#else
-  std::tm *brokenTime = ::localtime_r(&local, &tm);
-  if (!brokenTime) {
-    // Local time is invalid.
-    return 0;
-  }
-#endif
-  return tm.tm_isdst ? MS_PER_HOUR : 0;
-}
-
 //===----------------------------------------------------------------------===//
 // ES5.1 15.9.1.9
 

unittests/VMRuntime/DateUtilTest.cpp

@@ -145,6 +145,11 @@ double localTime(double t) {
 double utcTime(double t) {
   return hermes::vm::utcTime(t, getLocalTimeOffsetCache());
 }
+
+/// A wrapper that implements the deleted daylightSavingTA() function.
+double daylightSavingTA(double timeMs) {
+  return getLocalTimeOffsetCache().daylightSavingOffsetInMs(timeMs);
+}
 } // namespace
 
 TEST(DateUtilTest, LocalTZATest) {