Closed
Bug 162144
Opened 23 years ago
Closed 21 years ago
nsIFilePicker.show can throw exceptions
Categories
(Other Applications Graveyard :: Venkman JS Debugger, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: rginda)
References
()
Details
Attachments
(1 file)
|
1.46 KB,
patch
|
rginda
:
review+
|
Details | Diff | Splinter Review |
please change your show() callers to handle the possibilty that show() throws an
exception (important) and please don't access the file field in the case that
the user clicks cancel (trivial).
I'd write a patch but i can't think of how to write it to match the style of
your code.
|
||
Comment 1•21 years ago
|
||
|
Assignee | |
Comment 2•21 years ago
|
||
Comment on attachment 145784 [details] [diff] [review]
Simple patch to catch exceptions
r=rginda
Attachment #145784 -
Flags: review+
|
|
||
Comment 3•21 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Product: Core → Other Applications
|
||
Updated•6 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•