Its implementation is just wrong: it reads one character at a time,
because nsIBinaryOutputStream::WriteStringZ's implementation writes a
nul-terminated string. I plan to change both to stream a counted
string. I see the old cache using these methods
(http://lxr.mozilla.org/mozilla/source/netwerk/cache/mgr/nsCachedNetData.cpp#493,
http://lxr.mozilla.org/mozilla/source/netwerk/cache/mgr/nsCachedNetData.cpp#812),
but if those are the only uses, I argue we should break API
compatibility here and get rid of this single-byte-read nonsense. Comments?
/be
