265025 - Should use try/catch to see if reveal is supported

Status: RESOLVED DUPLICATE of bug 472001

(SeaMonkey :: Download & File Handling, defect)

Description

[[e-mail]]

The download manager (and download progress dialog) currently uses a platform
test to guess whether nsILocalFile.reveal is supported. Instead it should use
try/catch and if the reveal fails it can fall back to opening a browser
directory listing. It also means that should a future Unix port implement reveal
then it will automatically get used. See also attachment 162154 [details] [diff] [review] (FireFox).

Comment 1

[Tony Mechelynck [:tonymec]]

Hello Neil. Any idea if this old bug is still valid, and if it is, who (or among whom) would be the man (well, the "person") to fix it?

Comment 2

[[e-mail]]

(In reply to comment #1)
> Hello Neil. Any idea if this old bug is still valid, and if it is, who (or
> among whom) would be the man (well, the "person") to fix it?
Code to implement .reveal for Linux was added to nsILocalFile in the 1.9 timeframe so any new download manager should use .reveal on all platforms.

Comment 3

[Robert Kaiser]

As comment #2 states, this should be solved with the new download manager introduced by bug 472001