Comment 84 for bug 239826

Revision history for this message
In , Gavin Sharp (gavin-sharp) wrote :

(In reply to comment #59)
> > as long as there's suitable fallback (might want to remove the reportError...).
>
> FWIW, reportError() is what browser code uses for that case.

Right, but that pref is set in browser's default prefs. My point was that in the case of a generic toolkit app that doesn't set the pref, we might actually want to treat this like an ignorable failure. I didn't think through what that would mean, though (user clicks a button and nothing happens), so just ignore that comment - the reportError() is fine.