Bug 5930 - Further dom/* cleanup, also removing extra forwarding headers
Summary: Further dom/* cleanup, also removing extra forwarding headers
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P4 Normal
Assignee: Eric Seidel (no email)
URL:
Keywords:
Depends on: 5929
Blocks:
  Show dependency treegraph
 
Reported: 2005-12-04 05:11 PST by Eric Seidel (no email)
Modified: 2005-12-04 15:12 PST (History)
0 users

See Also:


Attachments
An extremely cleanup patch (32.53 KB, patch)
2005-12-04 05:13 PST, Eric Seidel (no email)
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2005-12-04 05:11:20 PST
Further dom/* cleanup, also removing extra forwarding headers

This continues work started with 5929.

A very simple patch is attached.
Comment 1 Eric Seidel (no email) 2005-12-04 05:13:58 PST
Created attachment 4933 [details]
An extremely cleanup patch

Anyone could review this extremely simple cleanup patch (just removes code).
Comment 2 Eric Seidel (no email) 2005-12-04 05:14:10 PST
Comment on attachment 4933 [details]
An extremely cleanup patch

Anyone could review this extremely simple patch.
Comment 3 Geoffrey Garen 2005-12-04 11:40:47 PST
Comment on attachment 4933 [details]
An extremely cleanup patch

rs=me (heh, always wanted to say that), but why remove the comments? no longer
accurate? documentation elsewhere?
Comment 4 Eric Seidel (no email) 2005-12-04 13:13:50 PST
Comment on attachment 4933 [details]
An extremely cleanup patch

The comments died because the whole files are really just living dead, they're
just limping along for now because they have necessary constats.  kdom, for
instance has moved these constants into other files as we will eventually to I
imagine.