Line 48 of nsMimeTypes.h has: #define APPLICATION_HTTP_INDEX "application/http-index" Neither the define nor the string are used anywhere in mozilla - all places which refer to the mime/type for an index file format use application/http-index-format, as does the spec at http://www.area.com/~roeber/file_format.html. The define should be changed to match the spec and the usage within mozilla. net.h also has the same problem.
http bugs to "Networking::HTTP"
nominating for moz 0.9
changing milestone to FUTURE
sr=darin, but consider these comments: 1) "mozilla/include/net.h" is dead. this is part of the old netlib. there shouldn't be any consumers of this file. 2) there are many other places in necko which hard code "http-index-format" what about making these use HTTP_INDEX_FORMAT as well? fgrep http-index-format `find . -name '*.cpp'` from mozilla/netwerk/ reveals the following set of files which should also be fixed: ./base/src/nsFileStreams.cpp ./cache/filecache/nsDiskCacheRecordChannel.cpp ./protocol/file/src/nsFileChannel.cpp ./protocol/ftp/src/nsFtpConnectionThread.cpp ./protocol/ftp/src/nsFtpConnectionThread.cpp ./protocol/ftp/src/nsFtpConnectionThread.cpp ./streamconv/converters/nsFTPDirListingConv.cpp ./streamconv/converters/nsGopherDirListingConv.cpp it'd be greatly appreciated if you would fix these as well. thanks!
I know net.h is obsolete, but I fixed it anyway just in case someone else was including it somewhere. The patches to nsFtpConnectionThread are attached to bug 68651. I'm also reassigning this to me so that I can find it easily when I get arround to applying for CVS write access :)
jag checked this in for me.
Verified per Bradley's comment