regChrome pointing to .jar w/o manifest incorrectly returns 0.

RESOLVED WONTFIX

Status

P3
normal
RESOLVED WONTFIX
18 years ago
3 years ago

People

(Reporter: depman1, Assigned: dveditz)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
builds 2000-09-29-09-MN6. cross platform.
1. Go to http://jimbob/trigger3.html
2. Enter http://jimbob/jars/a_reg_chrome_2_skin_jar.xpi in URL field. This 
points to classic.jar which doesn't contain a manifest.rdf file. (I also tried 
this test with my own .jar file which only contains a regular .txt file).
3. Press Trigger. Then OK.
4. Check log file.
Result: regChrome returns 0.
Expected: Should return an error since the .jar doesn't contain a manifest at 
the root level.

log file:

-------------------------------------------------------------------------------
file:///C|/XPIStage/test3/test3.xpi  --  09/29/2000 16:23:11
-------------------------------------------------------------------------------

     acceptance: a_reg_chrome_2_skin_jar.xpi
     ---------------------------------------

     ** returned getfolder = C:\Program Files\Netscape\M18\sep29-mN6\Netscape 
6\chrome\test.jar
     ** registerChrome of skins should return 0 = 0
     [1/1]	Register Skin: jar:file://C|/Program 
Files/Netscape/M18/sep29-mN6/Netscape 6/chrome/test.jar!/

     Install completed successfully
     Finished Installation  09/29/2000 16:23:11

script:

var regNode = "a_reg_chrome_2_skin_jar";
var vi = "1.1.1.1";
var jarFile = "classic.jar";

initInstall("acceptance: a_reg_chrome_2_skin_jar.xpi", regNode, vi, 0);

f = getFolder("Chrome", jarFile);
logComment("returned getfolder = " + f);
err = registerChrome(SKIN, f); 
logComment("registerChrome of skins should return 0 = " + err);

if (0 == getLastError())
   performInstall();
else
   cancelInstall();
(Reporter)

Comment 1

18 years ago
changed qa contact to depstein. note that this test uses 2 params for regChrome. 
will try out 3rd param for contents.rdf tests.
QA Contact: jimmylee → depstein
(Reporter)

Comment 2

18 years ago
.xpi renamed. Now run http://jimbob/jars/f_reg_chrm_2_skin_jar.xpi

Comment 3

18 years ago
Updating QA Contact.
QA Contact: depstein → jimmylee
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 4

11 years ago
The xpinstall script engine has been removed from the trunk, bugs in it are obsolete.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.