See e.g. http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1210055040.1210058195.22882.gz&fulltext=1 It's been failing intermittently since it was landed (bug 431729). Could be a timing issue, given that it uses a setTimeout. I've disabled it for now.
shouldn't be a timeout issue since it uses a setTimeout of zero so the load handlers fire
A setTimeout of 0 is a setTimeout of 10ms, and it can still introduce races if the machine is heavily bogged down (which is the case for the Linux unit test machines).
Summary: browser_sanitize-download-history.js failing intermittently on Linux tinderbox → Remove browser_sanitize-download-history.js
Created attachment 8540168 [details] [diff] [review] The patch This test has been disabled for quite some time, and it cannot be re-enabled because it uses nsIDownloadManager. There is no point in having the file in the tree. Other existing tests, rather than one with an intermittent failure, are a better starting points if we ever want to write a front-end test for this functionality.
Assignee: nobody → paolo.mozmail
Status: NEW → ASSIGNED
Attachment #8540168 - Flags: review?(mak77)
This is part of the work to remove all references to nsIDownloadManager.
Points: --- → 1
Attachment #8540168 - Flags: review?(mak77) → review+
strange, I was sure to have marked this patch yesterday... somehow my changes were not submitted...
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
You need to log in before you can comment on or make changes to this bug.