652192 - Remove layout/html

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

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.

Comment 1

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Diffstat

Patch is too big to attach, so I'll just ask for review on the diffstat

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

So we're giving up on HTML finally?  Woo!

On a serious note, why aren't we converting these to automated tests?

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

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.

Comment 4

[David Baron :dbaron:]

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 5

[David Baron :dbaron:]

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.

Comment 6

[David Baron :dbaron:]

Give other people a few days to object, though.  (Maybe send email to dev-tech-layout too?)

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Works for me.

Comment 8

[Bernd]

I share Davids wish to convert them but the source can be easily retrieved otherwise. r+

Comment 9

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Didn't get any comments on the newsgroup, so I went ahead:

http://hg.mozilla.org/mozilla-central/rev/5991f0ec6b34