The use of NS_ERROR_NO_CONTENT() from nsMailtoChannel::AsyncOpen() has caused problems when loading mailto: urls from the URILoader. It looks like the docshell code was patched to handle this error condition but the URI Loader hasn't been. Patch to follow.
I need review/super-review from you guys since this seems to be your area. I need this for X Remote fixes.
oops, you're right... I think that a better patch would be to combine the check with NS_ERROR_DOM_RETVAL_UNDEFINED. something like this: if (rv == NS_ERROR_DOM_RETVAL_UNDEFINED || rv == NS_ERROR_NO_CONTENT) rather than adding another if statement Hopefully, NS_ERROR_DOM_RETVAL_UNDEFINED is going away in favor of NS_ERROR_NO_CONTENT.
No longer blocks: 98603
What about the warning after the check? It's not a warning condition, is it?
Yeah... we can lose the NS_WARNING(...).
Comment on attachment 48476 [details] [diff] [review] patch #2 sr=mscott
Attachment #48476 - Flags: superreview+
Checked in. Thanks, guys!
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.