Closed Bug 54769 Opened 24 years ago Closed 17 years ago

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

Categories

(Core Graveyard :: Installer: XPInstall Engine, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: depman1, Assigned: dveditz)

Details

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();
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
Updating QA Contact.
QA Contact: depstein → jimmylee
Status: NEW → ASSIGNED
The xpinstall script engine has been removed from the trunk, bugs in it are obsolete.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.