Set the TZ env to get the UTC-ms since epoch.

runtime/vm/os_win.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/os.h"
 
 #include <time.h>
 
 #include "platform/assert.h"
 
 namespace dart {
 
 bool OS::BreakDownSecondsSinceEpoch(time_t seconds_since_epoch,
                                     bool in_utc,
                                     BrokenDownDate* result) {
   struct tm tm_result;
   errno_t error_code;
   if (in_utc) {
     error_code = gmtime_s(&tm_result, &seconds_since_epoch);
   } else {
-    // TODO(floitsch): we should be able to call tzset only once during
-    // initialization.
-    tzset();  // Make sure the libc knows about the local zone.

[cshapiro, 2012/04/19 23:52:12]

Why was this removed?  The documentation states that localtime consults the TZ
environment variable

http://msdn.microsoft.com/en-us/library/a442x3ye(v=vs.80).aspx

Either this call is needed for all conversions using the C library routines, or
it is not.

[floitsch, 2012/04/20 16:09:57]

localtime_s only looks at the env variable. tzset *uses* the env variable but
doesn't set it. Actually I think that localtime_s invokes tzset itself (quoting
the doc):
When TZ is set, three other environment variables (_timezone, _daylight, and
_tzname) are automatically set as well.

     error_code = localtime_s(&tm_result, &seconds_since_epoch);
   }
   result->year = tm_result.tm_year;
   result->month= tm_result.tm_mon;
   result->day = tm_result.tm_mday;
   result->hours = tm_result.tm_hour;
   result->minutes = tm_result.tm_min;
   result->seconds = tm_result.tm_sec;
   return error_code == 0;
 }
 
 
 bool OS::BrokenDownToSecondsSinceEpoch(
     const BrokenDownDate& broken_down, bool in_utc, time_t* result) {
   struct tm tm_broken_down;
   // mktime takes the years since 1900.
   tm_broken_down.tm_year = broken_down.year;
   tm_broken_down.tm_mon = broken_down.month;
   tm_broken_down.tm_mday = broken_down.day;
   tm_broken_down.tm_hour = broken_down.hours;
   tm_broken_down.tm_min = broken_down.minutes;
   tm_broken_down.tm_sec = broken_down.seconds;
   // Set wday to an impossible day, so that we can catch bad input.
   tm_broken_down.tm_wday = -1;
   // Make sure the libc knows about the local zone.
-  // In case of 'in_utc' this call is mainly for multi-threading issues. If
-  // another thread uses a time-function it will set the timezone. The timezone
-  // adjustement below would then not work anymore.
   // TODO(floitsch): we should be able to call tzset only once during
   // initialization.
   tzset();

[cshapiro, 2012/04/19 23:52:12]

Why is this being called before a conversion for a timezone independent value? 
This looks like it should precede the mktime call, only.

[floitsch, 2012/04/20 16:09:57]

Looking through the docs it looks as calling tzset should never be necessary.
Removing.

   if (in_utc) {
     // Disable daylight saving in utc mode.
     tm_broken_down.tm_isdst = 0;
-    // mktime assumes that the given date is local time zone.
-    *result = mktime(&tm_broken_down);
-    // Remove the timezone.
-    *result -= timezone;
+    *result = _mkgmtime(&tm_broken_down);
   } else {
     // Let libc figure out if daylight saving is active.
     tm_broken_down.tm_isdst = -1;
     *result = mktime(&tm_broken_down);
   }
-  if ((*result == -1) && (tm_broken_down.tm_wday == -1)) {
+  if ((*result == -1 || *result == 1) && (tm_broken_down.tm_wday == -1)) {

[cshapiro, 2012/04/19 23:52:12]

How is returning 1 an error?

[floitsch, 2012/04/20 16:09:57]

Couldn't find any reference to it either. Probably added before I switched to
_mkgmtime. I will remove it.

     return false;
   }
   return true;
 }
 
 
 int64_t OS::GetCurrentTimeMillis() {
   return GetCurrentTimeMicros() / 1000;
 }
 
@@ skipping 129 matching lines @@
 void OS::Abort() {
   abort();
 }
 
 
 void OS::Exit(int code) {
   exit(code);
 }
 
 }  // namespace dart