Closed Bug 432425 Opened 12 years ago Closed 6 years ago

Remove browser_sanitize-download-history.js

Categories

(Firefox :: General, defect)

defect
Not set
normal
Points:
1

Tracking

()

RESOLVED FIXED
Firefox 37
Iteration:
37.3 - 12 Jan

People

(Reporter: Gavin, Assigned: Paolo)

References

Details

Attachments

(1 file)

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
Attached patch The patchSplinter Review
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.
Blocks: 851471
Points: --- → 1
Flags: qe-verify-
Flags: firefox-backlog+
Iteration: --- → 37.2
Iteration: 37.2 → 37.3
Attachment #8540168 - Flags: review?(mak77) → review+
strange, I was sure to have marked this patch yesterday... somehow my changes were not submitted...
https://hg.mozilla.org/mozilla-central/rev/5ff375efac3c
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
You need to log in before you can comment on or make changes to this bug.