294302 - load() method in spidermonkey throws uncatchable error when called for non-existant file

Status: VERIFIED FIXED

(Core :: JavaScript Engine, defect, P4)

Description

[Alex Russell]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.8) Gecko/20050511 Firefox/1.0.4
Build Identifier: JavaScript-C 1.5

obelisk:~/dojo/tmp alex$ cat load_test.js 
try{
        load("nonexistant.js");
}catch(e){
        print("failed to load nonexistant.js with error: "+e);
}
print("foo");
obelisk:~/dojo/tmp alex$ cat load_test.js 
try{
        load("nonexistant.js");
}catch(e){
        print("failed to load nonexistant.js with error: "+e);
}
print("foo");
obelisk:~/dojo/tmp alex$ js load_test.js 
load_test.js:2: can't open nonexistant.js: No such file or directory


Reproducible: Always

Steps to Reproduce:
1. start spidermonkey shell
2. call load() function with non-existant file/URI as argument
3. attempt to continue after catching the thrown exception

Actual Results:  
unable to continue after catch handler called.

Expected Results:  
if exception is caught (and not re-thrown), execution should continue after the
catch() block is processed

Comment 1

[Gervase Markham [:gerv]]

This is an automated message, with ID "auto-resolve01".

This bug has had no comments for a long time. Statistically, we have found that
bug reports that have not been confirmed by a second user after three months are
highly unlikely to be the source of a fix to the code.

While your input is very important to us, our resources are limited and so we
are asking for your help in focussing our efforts. If you can still reproduce
this problem in the latest version of the product (see below for how to obtain a
copy) or, for feature requests, if it's not present in the latest version and
you still believe we should implement it, please visit the URL of this bug
(given at the top of this mail) and add a comment to that effect, giving more
reproduction information if you have it.

If it is not a problem any longer, you need take no action. If this bug is not
changed in any way in the next two weeks, it will be automatically resolved.
Thank you for your help in this matter.

The latest beta releases can be obtained from:
Firefox:     http://www.mozilla.org/projects/firefox/
Thunderbird: http://www.mozilla.org/products/thunderbird/releases/1.5beta1.html
Seamonkey:   http://www.mozilla.org/projects/seamonkey/

Comment 2

[Alex Russell]

(In reply to comment #1)
> This is an automated message, with ID "auto-resolve01".

This bug blocks serious command-line development on top of spidermonkey. Brendan Eich confirmed the 
issue at the Ajax Summit some months ago, but I'm not away of a fix to date.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Confirming and taking (I'll try to investigate this sometime soon).

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Allow round tripping

Until now JSMSG_CANT_OPEN could not be turned into an exception, and was therefore uncatchable.

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Real patch this time

Comment 6

[Brendan Eich [:brendan]]

Comment on attachment 201697 [details] [diff] [review]
Real patch this time

Hey, feel free to fix other JSEXN_NONE copy/paste bugs!

/be

Comment 7

[Blake Kaplan (:mrbkap) (inactive)]

I'll leave the other ones for another time. Do we want this in js1.6 (not necessarily Firefox 1.5)?

Checked into trunk.

Comment 8

[Bob Clary [:bc] (inactive)]

/cvsroot/mozilla/js/tests/js1_5/Regress/regress-294302.js,v  <--  regress-294302.js
initial revision: 1.1

Comment 9

[Bob Clary [:bc] (inactive)]

verified trunk.

Comment 10

[Bob Clary [:bc] (inactive)]

fixed by Bug 336373 on the 1.8.1 branch. 
verified fixed 1.8.1 with windows/macppc/linux 20060707