Components.utils.import reports NS_ERROR_FAILURE when the file not exists

RESOLVED FIXED in mozilla1.9alpha6

Status

()

--
trivial
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: asqueella, Assigned: asqueella)

Tracking

Trunk
mozilla1.9alpha6
x86
Windows XP
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

12 years ago
Is there a reason to return the generic error code?

Comment 1

12 years ago
(In reply to comment #0)
> Is there a reason to return the generic error code?
> 

Not that I know of, want to change it?
(Assignee)

Comment 2

12 years ago
Created attachment 266282 [details] [diff] [review]
patch
Assignee: nobody → asqueella
Status: NEW → ASSIGNED
Attachment #266282 - Flags: review?(sayrer)

Updated

12 years ago
Attachment #266282 - Flags: superreview?(brendan)
Attachment #266282 - Flags: review?(sayrer)
Attachment #266282 - Flags: review+

Updated

12 years ago
Attachment #266282 - Flags: superreview?(brendan) → superreview+
(Assignee)

Updated

12 years ago
Whiteboard: [checkin needed]
Target Milestone: --- → mozilla1.9alpha6
(Assignee)

Comment 3

12 years ago
Created attachment 267639 [details] [diff] [review]
as checked in

Needed a fix for do_get_file to support inexistent files (I had the line that checked for existence commented out in my head.js).

mozilla/js/src/xpconnect/loader/mozJSComponentLoader.cpp  1.135
mozilla/js/src/xpconnect/tests/unit/test_bogus_files.js   1.2
mozilla/tools/test-harness/xpcshell-simple/head.js        1.13
Attachment #266282 - Attachment is obsolete: true
(Assignee)

Updated

12 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [checkin needed]
You need to log in before you can comment on or make changes to this bug.