Olli, do you think maybe we should just remove this assertion? The code clearly works on platforms where TimeStamp is stable.
I guess so. It is still worrisome that we don't have reliable timestamps on Windows. We rely on timestamps comparisons everywhere, and all the time even more.
We do (should have) after bug 1345939, fixed a month ago. But it could be that the VM still doesn't provide it in a stable way and we have no way to detect this, relying on the windows API to cover for us. I would be really interested in what the timestamps in test values are when this assertion fails.
1 failures in 720 pushes (0.001 failures/push) were associated with this bug in the last 7 days. Repository breakdown: * autoland: 1 Platform breakdown: * windows7-32: 1 For more details, see: https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1380518&startday=2017-07-10&endday=2017-07-16&tree=all
Priority: -- → P3
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.