38989 - Triggering xpis does not install and freezes seamonkey (crash)

Status: VERIFIED FIXED

(Core :: XUL, defect, P1)

Description

[Jimmy Lee]

Build: 2000-05-11-08-M16(MAC)

1. From http://jimbob/trigger3.html, trigger any xpi file

RESULT:
No confirmation dialog appears.  Clicking on menus does not drop down menu 
items.  Clicking on browser buttons does nothing.  Force quit is necessary.

EXPECTED RESULT:
Confirmation dialog appears.  No application lock ups.

Comment 1

[Jimmy Lee]

Adding pp to keywords.  Linux and Windows are able to trigger xpis.

Comment 2

[Cathleen]

samir, do you have a cycle to take a look at this bug today?

apparently, triggering any xpi on the mac is blocked... no testing can be done. 
possibly blocked by the UI dialogs stuff.

Comment 3

[Samir Gehani]

nsIDOMWindow::OpenDialog hangs when called from XPInstall at:
http://lxr.mozilla.org/seamonkey/source/xpinstall/src/nsXPInstallManager.cpp#267

Our xul has not changed since Apr 24, 2000 so that rules out a xul problem: 
(http://lxr.mozilla.org/seamonkey/source/xpinstall/res/content/institems.xul)

Test case:
----------
1> Navigate to: http://puma/xpinstall/test/sanity_trigger.html
2> Set break point at line 267 in nsXPInstallManager where OpenDialog is called.
3> Click on "Trigger Sanity Zippy" link on page.
4> OpenDialog doesn't return.

NOTE: This is mac-specific.

Cathleen will be tracking down the appropriate folks to look at this.

Comment 4

[Samir Gehani]

My updated mac build just finished and I still see the problem which rules out 
the suggestion by some that this is fixed already.

Comment 5

[Samir Gehani]

Reassigning to Dan Matejka per Cathleen.

Comment 6

[Samir Gehani]

Thanks Dan :o)

Comment 7

[Peter Trudelle]

[nsbeta2+]

Comment 8

[Jimmy Lee]

Ccing Dan.

Comment 9

[Jimmy Lee]

Adding "crash" to Summary since a force quit is neccessary.

Comment 10

[leger]

This bug has blocked XPInstall engine testing since 5/11/00.  We have not been 
able to do any Macintosh testing since this date.  Without this fix, XPInstall 
does not have a M16 XPInstall solution.  

I am putting this on [dogfood+]...can we please get a fix ASAP?

Comment 11

[leger]

Adding crash keyword

Comment 12

[Gervase Markham [:gerv]]

Peter Trudelle commented "nsbeta2+" yet the Status Whiteboard claims nsbeta2-. 
Seeing as this bug is dogfood+, and therefore has to be fixed for M16, which is 
probably before nb2b, I am resolving this in what seems like the sensible way, 
by changing nsbeta2- to nsbeta2+.

If this is a problem, feel free to adjust it :-)

Gerv

Comment 13

[Dan M]

xpinstall.css was missing.

Comment 14

[Jimmy Lee]

Build: 2000-05-26-08-M16(MAC)

Back to normal.  Thanks all.