Closed Bug 610185 Opened 9 years ago Closed 9 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)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla5
Tracking Status
status2.0 --- .1-fixed

People

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

References

(Blocks 1 open bug)

Details

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

Attachments

(2 files)

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: 9 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.