REGRESSION(r120552): Many tests timing out on Windows
authorjhoneycutt@apple.com <jhoneycutt@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Jun 2012 09:12:19 +0000 (09:12 +0000)
committerjhoneycutt@apple.com <jhoneycutt@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Jun 2012 09:12:19 +0000 (09:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=89433

Reviewed by Andy Estes.

* wtf/CurrentTime.cpp:
(WTF::lowResUTCTime):
It looks like there were two bugs:
1) the function was converting from 100ths of nanoseconds to seconds
   rather than 100s of nanoseconds to seconds by dividing by
   (nsPerSecond * 100) rather than (nsPerSecond / 100)
2) the function should've been returning milliseconds, not seconds

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@120697 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/CurrentTime.cpp

index 8bb695f..6a06e89 100644 (file)
@@ -1,3 +1,18 @@
+2012-06-19  Jon Honeycutt  <jhoneycutt@apple.com>
+
+        REGRESSION(r120552): Many tests timing out on Windows
+        https://bugs.webkit.org/show_bug.cgi?id=89433
+
+        Reviewed by Andy Estes.
+
+        * wtf/CurrentTime.cpp:
+        (WTF::lowResUTCTime):
+        It looks like there were two bugs:
+        1) the function was converting from 100ths of nanoseconds to seconds
+           rather than 100s of nanoseconds to seconds by dividing by
+           (nsPerSecond * 100) rather than (nsPerSecond / 100)
+        2) the function should've been returning milliseconds, not seconds
+
 2012-06-18  Tony Payne  <tpayne@chromium.org>
 
         [chromium] Add iccjpeg and qcms to chromium port.
index 884f1ba..3256309 100644 (file)
@@ -71,7 +71,7 @@ namespace WTF {
 
 // Number of 100 nanosecond between January 1, 1601 and January 1, 1970.
 static const ULONGLONG epochBias = 116444736000000000ULL;
-static const double nsPerSecond = 1000000000.0;
+static const double hundredsOfNanosecondsPerMillisecond = 10000;
 
 static double lowResUTCTime()
 {
@@ -91,7 +91,7 @@ static double lowResUTCTime()
     memcpy(&dateTime, &fileTime, sizeof(dateTime));
 
     // Windows file times are in 100s of nanoseconds.
-    return (dateTime.QuadPart - epochBias) / (100 * nsPerSecond);
+    return (dateTime.QuadPart - epochBias) / hundredsOfNanosecondsPerMillisecond;
 }
 
 #if USE(QUERY_PERFORMANCE_COUNTER)