Closed Bug 652192 Opened 9 years ago Closed 9 years ago
Having non-automated tests that nobody runs only serve to confuse people, and they will be in the history if anybody ever wants to do something about it.
Patch is too big to attach, so I'll just ask for review on the diffstat
Attachment #527833 - Flags: review?(dbaron)
So we're giving up on HTML finally? Woo! On a serious note, why aren't we converting these to automated tests?
It would certainly be nice if someone stepped up to do that, but I don't see it happening. In any case, if such a hero would appear, he can always find these tests in the history.
The reason I've been hesitant to delete them in the past is that I do want them to be converted to automated tests...
Comment on attachment 527833 [details] Diffstat I suppose there's not particularly more value in converting the things that happened to be checked in here than anything else in bugzilla, so r=dbaron.
Attachment #527833 - Flags: review?(dbaron) → review+
Give other people a few days to object, though. (Maybe send email to dev-tech-layout too?)
Works for me.
I share Davids wish to convert them but the source can be easily retrieved otherwise. r+
Didn't get any comments on the newsgroup, so I went ahead: http://hg.mozilla.org/mozilla-central/rev/5991f0ec6b34
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.