evaluate performance of mozilla cookies implementation. in particular, i'd like
to determine how expensive it is to be calling into the cookie manager for every
here's some results using a linux commercial build from 2001/10/01:
Test id: 3BC15AD47D
Avg. Median : 1060 msec Minimum : 252 msec
Average : 1092 msec Maximum : 3812 msec
Test id: 3BC15CFDD8
Avg. Median : 1027 msec Minimum : 247 msec
Average : 1051 msec Maximum : 3826 msec
this amounts to a 3% performance difference, and i think it means that we
should not be sending notifications to the cookie manager unless we actually
have cookies to report. see bug 96178 for existing plans to make this possible.
morse: this bug might interest you.
something along the lines of bug 96178 might help.
-> 0.9.8 (perhaps even later)
unlikely to get around to doing anything about this in the mozilla1.0 timeframe.
Is this bug still relevent now that the cookie code is being rewritten?
sort of.. this is more of a meta bug than anything else. the point of this bug
was to determine why the presence of the cookie DLL was so costly. we may or
may not have addressed all of that with the cookie rewrite. i doubt it since we
didn't see much of any change in Tp with the current cookie changes :-/
ah, thx for cc'ing me darin.
"cookies" and "performance" do not go in the same sentence, unless there's a
strong negation present. the rewrite has not done anything to address
performance yet (apart from micro optimizations and other fun stuff).
the perf improvements will be coming up shortly. phase 3 sets the stage for
this, and I may introduce arenas there (they're so easy) so that'll give us some
malloc bloat + perf wins.
the really big win will come when
a) we move to hashes (phase 4, dare i mention 1.4b in this sentence? ;)
darin: i've actually just finished coding up the hash conversion. so do let me
know if there's a chance of it being on the radar for 1.4b, and i'll push it
through ASAP. not sure on this one because although it's a conceptually simple
change, it is a big patch and i'm not sure what your take will be for 1.4b.
b) we integrate cookies into necko and kill that infernal nsCookieHTTPNotify.
dwitte: any update?
Should this bug still be opened?
Nope. Cookies are fast now!