220463 - use more error codes in nsExternalAppHandler::SendStatusChange

Status: RESOLVED FIXED

(Core Graveyard :: File Handling, defect)

Description

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

bug 220406 added some more error codes returned from nsLocalFileWin::Launch
some are already caught in SendStatusChange, some are not yet. this bug is filed
for them.

Comment 1

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

Attachment: patch

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 132278 [details] [diff] [review]
patch

r=bzbarsky

Comment 3

[Darin Fisher]

Comment on attachment 132278 [details] [diff] [review]
patch

hmm... but will there be enough memory to throw up the error dialog? ;-)

sr=darin

Comment 4

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

heh :) well, there's nothing we can do if there isn't.

Checking in
embedding/components/webbrowserpersist/locale/en-US/nsWebBrowserPersist.properties;
/cvsroot/mozilla/embedding/components/webbrowserpersist/locale/en-US/nsWebBrowserPersist.properties,v
 <--  nsWebBrowserPersist.properties
new revision: 1.4; previous revision: 1.3
done
Checking in uriloader/exthandler/nsExternalHelperAppService.cpp;
/cvsroot/mozilla/uriloader/exthandler/nsExternalHelperAppService.cpp,v  <-- 
nsExternalHelperAppService.cpp
new revision: 1.223; previous revision: 1.222
done