Last Comment Bug 716961 - clean up HTTP header field name constants in nsHttpAtomList.h
: clean up HTTP header field name constants in nsHttpAtomList.h
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Networking: HTTP (show other bugs)
: unspecified
: All All
: -- trivial (vote)
: mozilla12
Assigned To: Julian Reschke
:
: Patrick McManus [:mcmanus]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-10 11:02 PST by Julian Reschke
Modified: 2012-01-11 18:18 PST (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Proposed patch (2.51 KB, patch)
2012-01-10 13:35 PST, Julian Reschke
bzbarsky: review+
Details | Diff | Splinter Review

Description Julian Reschke 2012-01-10 11:02:04 PST
The list of predefined header atoms contains several entries that are either not HTTP header fields (Content-Transfer-Encoding), not used (Status-URI), or both.

I think it would be best to remove all unused ones; at least when they aren't actual HTTP header fields.
Comment 1 Julian Reschke 2012-01-10 13:35:07 PST
Created attachment 587455 [details] [diff] [review]
Proposed patch
Comment 2 Boris Zbarsky [:bz] (still a bit busy) 2012-01-10 13:51:44 PST
Comment on attachment 587455 [details] [diff] [review]
Proposed patch

r=me
Comment 3 Patrick McManus [:mcmanus] 2012-01-10 17:51:47 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/d71926e38f2c
Comment 4 Patrick McManus [:mcmanus] 2012-01-10 17:52:38 PST
Thanks Julian!
Comment 5 Ed Morley [:emorley] 2012-01-11 18:18:56 PST
https://hg.mozilla.org/mozilla-central/rev/d71926e38f2c

Note You need to log in before you can comment on or make changes to this bug.