261091 - put child docloader's loadgroups into parent loadgroup

Status: NEW

(Firefox :: File Handling, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

nsDocLoaderImpl::Stop could be greatly simplified if its loadgroup would contain
the loadgroup of its child loaders. Then, it could just do
mLoadGroup->Cancel(NS_BINDING_ABORTED); and the child's groups would be
automatically cancelled.

care has to be taken not to
o) assume all children implement nsIChannel
o) only send state information to listeners if a channel gets added/removed, not
a loadgroup (what about dummy layout/parser requests?)
o) maybe other things...

Comment 1

[Christian :Biesinger (don't email me, ping me on IRC)]

in fact, maybe that method could be eliminated then, along with
nsURILoader::Stop(nsISupports* loadCookie)

Comment 2

[Boris Zbarsky [:bzbarsky]]

Once this is fixed, I think the code I added for bug 299553 can be removed...

Comment 3

[Boris Zbarsky [:bzbarsky]]

Also once this is fixed the code I'm adding for bug 330089 can be greatly simplified...

Comment 4

[Honza Bambas (:mayhemer)]

Preliminarily CC my self, I might need this for security UI.