Closed Bug 437422 Opened 13 years ago Closed 13 years ago
Automate litmus test for testing search and clear list
This should be a chrome test that lives in toolkit/mozapps/downloads/tests/chrome/ to replace this litmus test: https://litmus.mozilla.org/show_test.cgi?id=5285
Comment on attachment 328286 [details] [diff] [review] v1.0 This file suffers from lots of the same things mentioned in bug 443585 comment 2. Please fix and make a new attachment.
Attachment #328286 - Flags: review?(sdwilsh) → review-
Test updated to accommodate the new utils.js. The test passes
Comment on attachment 331482 [details] [diff] [review] v3.0 This test doesn't quite do what the litmus tests does. I've fixed it. Uploading shortly.
Attachment #331482 - Flags: review?(sdwilsh) → review-
Fixes the test and addresses review comments
Pushed to mozilla-central: http://hg.mozilla.org/mozilla-central/index.cgi/rev/46e2648da0fc Stephen - you can now disable this litmus test :)
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1a2
Litmus test disabled; w00t! (Thanks guys!)
stephen: Based on Comment 8, I assume we can remove the in-Litmus? flag which is currently set in this bug?
(In reply to comment #9) > stephen: Based on Comment 8, I assume we can remove the in-Litmus? flag which > is currently set in this bug? Yup; done, thanks!
Flags: in-litmus? → in-litmus-
You need to log in before you can comment on or make changes to this bug.