Closed Bug 1382071 Opened 8 years ago Closed 8 years ago

Intermittent dom/events/test/test_eventTimeStamp.html | Event timestamp (78) is in expected range: (78, 78)

Categories

(Core :: DOM: Events, defect, P5)

defect

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox59 --- fixed
firefox60 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: dholbert)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Priority: -- → P5
It looks like this comes from this code: > ok(evt.timeStamp > timeBeforeEvent && > evt.timeStamp < timeAfterEvent, > "Event timestamp (" + evt.timeStamp + ") is in expected range: (" + > timeBeforeEvent + ", " + timeAfterEvent + ")"); ...and we "fail" when all three timestamps are equal (since it's a strict less-than/greater-than comparison. Brian, looks like you wrote this test back in Bug 77992 -- can you confirm that this would be a reasonable change? (And: does this failure seem worrisome/broken to you, if you remember how this test works?)
Flags: needinfo?(bbirtles)
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Flags: needinfo?(bbirtles)
Comment on attachment 8947159 [details] Bug 1382071: Change a comparison in test_eventTimeStamp.html to consider its upper & lower bounds as part of the valid range. https://reviewboard.mozilla.org/r/216928/#review222760
Attachment #8947159 - Flags: review?(bbirtles) → review+
Pushed by dholbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c1c0bf22cf97 Change a comparison in test_eventTimeStamp.html to consider its upper & lower bounds as part of the valid range. r=birtles
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: