[Webkit-unassigned] [Bug 152828] Replace HTTPHeaderMap by HTTPHeaderList

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=152828

--- Comment #4 from youenn fablet <youennf at gmail.com> ---
> > I would first like to have a simple functional HTTPHeaderList with a clean
> > API. Then it could be optimized internally to handle large sets.
> This would be a performance regression until the optimizations are done.  I
> don't think we should do this.

I don't fully agree here.
Practically speaking, it would be a small performance improvement, except for one hypothetical case.
I am not against supporting this hypothetical case.
But there is a tradeoff here, the question is then how valuable is it to support it compared to complexity/maintenance cost?

In libsoup, the headers are stored as arrays.
Some headers are made specific (Content-Type, Content-Length) to be retrieved quickly.
IIRC, lazy optimization is also done to retrieve quickly combined header values.

Libcurl is lower-level so does not provide direct get/remove API but the principle is similar.

A list of header is good in the sense that it is fast to build and above layer can rearrange it for their own needs.