Created attachment 256567 [details] [diff] [review] Clean up the layout reftests Attached is a patch which sorts all of the layout reftests by bug number and alphabetically when necessary.
So in many cases the order was semantic -- starting with easier tests and moving to harder tests. I think the only ones that should be sorted are the bugs, and just by bug number (leaving the current order within bugs). And at the same time we should probably create a reftest.list in the bugs/ subdirectory and include it in the one above.
Created attachment 256738 [details] [diff] [review] Sort only bug reftests and move them to separate list Like so?
Attachment #256567 - Attachment is obsolete: true
Comment on attachment 256738 [details] [diff] [review] Sort only bug reftests and move them to separate list Yep, looks good. r+sr=dbaron. Do you need someone to land this for you?
Checked in to trunk. (I also added a comment at the top saying to keep it sorted by bug number.) Thanks for the patch.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
For the record, the object/ tests were sorted by the ordering found at their original location on biesi's site, so not rearranging them was the right thing to do. :-) I dislike opaque numbers, tho, so I tried to give them meaningful names to aid in guessing what they do if any one were to fail in the future.
and I did try to list simpler tests first, more complex tests later there.
You need to log in before you can comment on or make changes to this bug.