Closed Bug 610185 Opened 15 years ago Closed 14 years ago

Daylight Time failure in ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getTime() wrong value item 22

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla5
Tracking Flags:
Tracking Status
status2.0 --- .1-fixed

People

(Reporter: robert.strong.bugs, Assigned: bc)

References

Details

(Keywords: intermittent-failure, Whiteboard: [orange:time-bomb][fixed-in-tracemonkey])

Attachments

(2 files)

mozilla-central patch to mark test as fully random
833 bytes, patch
jorendorff
: review+
Details | Diff | Splinter Review
tracemonkey patch to mark test as fully random
779 bytes, patch
jorendorff
: review+
Details | Diff | Splinter Review
Rev3 MacOSX Leopard 10.5.8 mozilla-central debug test jsreftest on 2010/11/07 00:57:37 http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1289116657.1289118023.30053.gz REFTEST TEST-UNEXPECTED-FAIL | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getTime() wrong value item 22 REFTEST TEST-UNEXPECTED-FAIL | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.valueOf() wrong value item 23 REFTEST TEST-PASS | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getUTCFullYear() item 24 REFTEST TEST-PASS | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getUTCMonth() item 25 REFTEST TEST-PASS | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getUTCDate() item 26 REFTEST TEST-PASS | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getUTCDay() item 27 REFTEST TEST-UNEXPECTED-FAIL | file:///Users/cltbld/talos-slave/mozilla-central_leopard-o-debug_test-jsreftest/build/jsreftest/tests/jsreftest.html?test=ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getUTCHours() wrong value item 28
Whiteboard: [orange][orange:time-bomb]
Probably time to stop being merely suspicious.
Summary: jsreftest failure looking suspiciously like a Daylight Time failure: 15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getTime() wrong value item 22 → Daylight Time failure in ecma/Date/15.9.5.34-1.js | TDATE = new Date(TIME_NOW); (TDATE).setMonth(11,31); TDATE.getTime() wrong value item 22
I'll do a patch to disable them on tracemonkey and mozilla-central asap.
Attachment #519187 - Flags: review?(jorendorff)
Attachment #519188 - Flags: review?(jorendorff)
philor: I can't use http.oscpu in the jstests since the shell based jstests.py don't support it. It is easiest to just mark the test random until the errant subtests can be fixed and the xp machines upgraded to sp3.
ping review?
Attachment #519188 - Flags: review?(jorendorff) → review+
Attachment #519187 - Flags: review?(jorendorff) → review+
http://hg.mozilla.org/tracemonkey/rev/d1e18a84c3ce waiting for confirmation whether I need approval for fennec or gecko2.0 before checking into mozilla-central.
Whiteboard: [orange][orange:time-bomb] → [orange][orange:time-bomb][fixed-in-tracemonkey]
You do not: test-only patches can land without approval.
(In reply to comment #104) > You do not: test-only patches can land without approval. Ok. I'll land once the latest changeset finishes its tests.
(In reply to comment #106) > (In reply to comment #104) > > You do not: test-only patches can land without approval. > > Ok. I'll land once the latest changeset finishes its tests. Let me thank you for that in advance! This helps clean up the orange craziness that has been going on in m-c for the past few days.
fennec just went to rc, so i'm going to hold off landing on mc until things have settled down.
Depends on: post2.0
http://hg.mozilla.org/mozilla-central/rev/d0e5ba1e0bbb I'll dupe this to one of the outstanding dst bugs in a bit.
Bob, is there anything more to be done in this bug?
Ehsan: I don't think so. Birch appears to be the only remaining repo that hasn't synced. The underlying bug in the suite still remains though but that is handled in other bugs. Shall we mark this as fixed?
Can't make it any more noisy :)
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee: general → bclary
status2.0: --- → ?
Flags: in-testsuite+
Target Milestone: --- → mozilla2.2
Whiteboard: [orange][orange:time-bomb][fixed-in-tracemonkey] → [orange:time-bomb][fixed-in-tracemonkey]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: