nsLoadGroup::GetName does a lot of string copies when it doesn't need to do any. I'll attach a simple patch to fix this. The only thing it changes is that it causes the function to propagate the return value from mDefaultLoadRequest rather than always returning NS_OK.
Comment on attachment 51421 [details] [diff] [review] patch r=jag
Comment on attachment 51421 [details] [diff] [review] patch sr=darin
BTW... is this function actually called in optimized builds? i thought it was just used for debugging purposes.
I don't know. It was just too ugly for me to leave it untouched. (After all, people copy code from one place to another, and it could end up copied to somewhere performance is important...)
Fix checked in 2001-10-10 18:09 PDT.
david: can you mark this verified?