Reviewed by Darin.

        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=6915
        fast/js/date-constructor.html testcase is timezone-dependent

        * fast/js/date-constructor-expected.txt:
        * fast/js/resources/date-constructor.js:
        Disable the offending tests.

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

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 95b6eee9f0338b49b822aae792e4f6098cc1690e..de0428ae55abb1360706c251224355160bcbcff6 100644 (file)
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,14 @@
+2006-05-17  Alexey Proskuryakov  <ap@nypop.com>
+
+        Reviewed by Darin.
+
+        - test for http://bugzilla.opendarwin.org/show_bug.cgi?id=6915
+        fast/js/date-constructor.html testcase is timezone-dependent
+
+        * fast/js/date-constructor-expected.txt:
+        * fast/js/resources/date-constructor.js:
+        Disable the offending tests.
+
 2006-05-17  Eric Seidel  <eseidel@apple.com>
 
         Reviewed by andersca.

diff --git a/LayoutTests/fast/js/date-constructor-expected.txt b/LayoutTests/fast/js/date-constructor-expected.txt
index 52eee26753eed28cc206938a9d03a640350abac6..7bdb8d5a6df21df00e28c2c9e9bba04710aefcaf 100644 (file)
--- a/LayoutTests/fast/js/date-constructor-expected.txt
+++ b/LayoutTests/fast/js/date-constructor-expected.txt
@@ -23,8 +23,6 @@ PASS new Date(new Date(1111, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset is -2710
 PASS new Date(new Date(1111, 1, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset is -27104799538999
 PASS new Date(new Date(1111, 1, 1, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset is -27104799538999
 PASS new Date(new Date(1111, 1, 1, 1, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset is -27104799538999
-PASS new Date(1111, 1111, 1111, 1111, 1111, 1111, 1111, 1111).getTime() - timeZoneOffset is -28085817599889
-PASS new Date(new Date(1111, 1111, 1111, 1111, 1111, 1111, 1111, 1111)).getTime() - timeZoneOffset is -28085817599889
 PASS successfullyParsed is true
 
 TEST COMPLETE

diff --git a/LayoutTests/fast/js/resources/date-constructor.js b/LayoutTests/fast/js/resources/date-constructor.js
index d8f2d727b48c39946e74ae8c7415e50f3dc0535a..7b214630b281aa34515435524d123cabce0f79cc 100644 (file)
--- a/LayoutTests/fast/js/resources/date-constructor.js
+++ b/LayoutTests/fast/js/resources/date-constructor.js
@@ -33,10 +33,9 @@ shouldBe('new Date(new Date(1111, 1, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset'
 shouldBe('new Date(new Date(1111, 1, 1, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset', '-27104799538999');
 shouldBe('new Date(new Date(1111, 1, 1, 1, 1, 1, 1, 1, 1)).getTime() - timeZoneOffset', '-27104799538999');
 
-// I believe these next results may be incorrect.
-// Firefox gives different results.
-// Keep the tests here as a regression test for now.
-shouldBe('new Date(1111, 1111, 1111, 1111, 1111, 1111, 1111, 1111).getTime() - timeZoneOffset', '-28085817599889');
-shouldBe('new Date(new Date(1111, 1111, 1111, 1111, 1111, 1111, 1111, 1111)).getTime() - timeZoneOffset', '-28085817599889');
+// In Firefox, the results of the following tests are timezone-dependent, which likely implies that the implementation is not quite correct.
+// Our results are even worse, though, as the dates are clipped: (new Date(1111, 1201).getTime()) == (new Date(1111, 601).getTime())
+// shouldBe('new Date(1111, 1111, 1111, 1111, 1111, 1111, 1111, 1111).getTime() - timeZoneOffset', '-24085894227889');
+// shouldBe('new Date(new Date(1111, 1111, 1111, 1111, 1111, 1111, 1111, 1111)).getTime() - timeZoneOffset', '-24085894227889');
 
 var successfullyParsed = true;