Closed Bug 588151 Opened 9 years ago Closed 9 years ago
App Dlg doesn't do anything for Nothing
We never(?) handle the "Nothing" case in HelperAppDlg. We need to cancel the download.
Assignee: nobody → mark.finkle
Attachment #466750 - Flags: review?(mbrubeck)
Comment on attachment 466750 [details] [diff] [review] patch >+ const NS_BINDING_ABORTED = 0x804b0002; I'd rather see this line at the top of the function (or the file) instead of repeated twice... your call though, if you prefer it this way. (If you want to refactor this function a little to remove all the code duplication that'd be great too, but I'm not going to demand it.)
Attachment #466750 - Flags: review?(mbrubeck) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Verified on: Mozilla/5.0(Android; Linux armv7l; rv:2.0b5pre) Gecko/20100820 Namoroka/4.0b5pre Fennec/2.0a1pre
You need to log in before you can comment on or make changes to this bug.