87476 - Why have both nsContentDLF and nsLayoutDLF?

Status: RESOLVED DUPLICATE of bug 71248

(Core :: Layout, defect, P2)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Why do we have both nsContentDLF and nsLayoutDLF?  This looks like a mistake in
the layout/content split, I think.  It seems like we only use nsLayoutDLF. 
(This is confusing for people who want to modify the code -- both have been
modified recently.)

Comment 1

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I did not analyze these two files that much in the split - I just made the
minimal change to get things working.

Before you go and remove one or both, please make sure that whatever you do you
still leave in the possibility of creating a "DOM Builder" component that could
be used on the server, for example, without layout.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Changes made to nsContentDLF.cpp (unused) are:
C      L
1.2    1.41
1.3    ----
1.4    1.42
1.5    1.46
1.6    1.48
1.7    1.49
1.8    1.51

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The only revisions where the changes were different between the two were 1.3,
1.5, and 1.6, but I think there were some initial differences too.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

http://lxr.mozilla.org/seamonkey/source/xpfe/appshell/src/nsWebShellWindow.cpp#146
was confusing me about what we were using, but it's not used.

*** This bug has been marked as a duplicate of 71248 ***