Closed
Bug 395618
Opened 17 years ago
Closed 17 years ago
UseUserPrefFile should check that user.js exists
Categories
(Core :: Preferences: Backend, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: sayrer, Assigned: sayrer)
Details
Attachments
(1 file)
1.10 KB,
patch
|
benjamin
:
review+
benjamin
:
superreview+
benjamin
:
approval1.9+
|
Details | Diff | Splinter Review |
We try to open the file as an input stream before checking that it exists
Assignee | ||
Comment 1•17 years ago
|
||
Assignee: nobody → sayrer
Status: NEW → ASSIGNED
Attachment #280295 -
Flags: superreview?(benjamin)
Attachment #280295 -
Flags: review?(benjamin)
Assignee | ||
Comment 2•17 years ago
|
||
hmm, NS_ERROR_FILE_NOT_FOUND is better, wonder why I didn't see that when I looked in LXR.
Comment 3•17 years ago
|
||
Comment on attachment 280295 [details] [diff] [review] call exists() r=me with your nit
Attachment #280295 -
Flags: superreview?(benjamin)
Attachment #280295 -
Flags: superreview+
Attachment #280295 -
Flags: review?(benjamin)
Attachment #280295 -
Flags: review+
Attachment #280295 -
Flags: approval1.9+
Assignee | ||
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•