186719 - Audit NewURI implementations

Status: RESOLVED INCOMPLETE

(Core :: Networking, defect)

Description

[timeless]

so, in response to "-*- jsdHandler: aBaseURI passed to newURI, bailing." (no
longer in cvs), i decided to do a mini audit of NewURI impls. This is a meta
bug, it shouldn't live longer than a week.

Comment 1

[Mark Anderson]

Attachment: Patch for several of these bugs

This fixes the bugs that deal with SetSpec not being checked for failure, the
newURI Javascript issue (actually removing unnecessary checks), the NS_ADDREF
issues, and the memory leak.  More tomorrow.

Comment 2

[Mark Anderson]

If I'm taking this, it might as well be assigned to me.

Comment 3

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

uh, can you attach your patch to the more specific bugs that this bugs depends
on, splitting it up per component? meta bugs don't usually have patches attached
to them...

Comment 4

[Mark Anderson]

Well, I certainly can, but timeless implied that he wanted one patch to wipe
them all out.

Comment 5

[:aceman]

Hi. What is the state of this bug?
All of the blocking bugs are getting fixed (I am at the last two).
Will this meta bug be closed after those two?

Is the patch here still relevant?