78267 - Alert/Confirm dialogs called from xpi results in crash (Mac and Linux)

Status: VERIFIED FIXED

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

Description

[Jimmy Lee]

Build: 2001-04-30-08-trunk(LINUX)

1. From http://jimbob/trigger3.html, click on Functional drop-down and choose
   f_alertconfirm
2. Click Trigger case button
3. Click OK button from the confirm dialog
4. When alert appears, click OK button

RESULT:
Crash.

EXPECTED RESULT:
No crash.

NOTE:
Alert/confirms are ok when called from an html page 
(http://jimbob/bugs/confirm_it.html)

TalkBack Incident ID 29832799

Stack Trace
js_MarkGCThing() 
js_MarkGCThing() 
js_MarkGCThing() 
js_MarkGCThing() 
js_MarkGCThing() 
js_GC() 
js_ForceGC() 
js_DestroyContext() 
JS_DestroyContext() 
nsJSContext::~nsJSContext() 
nsJSContext::Release() 
nsCOMPtr_base::~nsCOMPtr_base() 
nsJSContext::CallEventHandler() 
nsJSEventListener::HandleEvent() 
nsEventListenerManager::HandleEventSubType() 
nsEventListenerManager::HandleEvent() 
nsXULElement::HandleDOMEvent() 
PresShell::HandleDOMEventWithTarget() 
nsButtonBoxFrame::MouseClicked() 
nsButtonBoxFrame::HandleEvent() 
PresShell::HandleEventInternal() 
PresShell::HandleEventWithTarget() 
nsEventStateManager::CheckForAndDispatchClick() 
nsEventStateManager::PostHandleEvent() 
PresShell::HandleEventInternal() 
PresShell::HandleEvent() 
nsView::HandleEvent() 
nsViewManager::DispatchEvent() 
HandleEvent() 
nsWidget::DispatchEvent() 
nsWidget::DispatchWindowEvent() 
nsWidget::DispatchMouseEvent() 
nsWidget::OnButtonReleaseSignal() 
nsWindow::HandleGDKEvent() 
dispatch_superwin_event() 
handle_gdk_event() 
libgdk-1.2.so.0 + 0x16ab2 (0x406afab2) 
libglib-1.2.so.0 + 0xf2c6 (0x406d92c6) 
libglib-1.2.so.0 + 0xf801 (0x406d9801) 
libglib-1.2.so.0 + 0xf8a3 (0x406d98a3) 
nsAppShell::DispatchNativeEvent() 
nsXULWindow::ShowModal() 
nsWebShellWindow::ShowModal() 
nsContentTreeOwner::ShowAsModal() 
nsWindowWatcher::OpenWindowJS() 
nsWindowWatcher::OpenWindow() 
nsPromptService::DoDialog() 
nsPromptService::Alert() 
XPTC_InvokeByIndex() 
EventHandler() 
PL_HandleEvent() 
PL_ProcessPendingEvents() 
nsEventQueueImpl::ProcessPendingEvents() 
event_processor_callback() 
our_gdk_io_invoke() 
libglib-1.2.so.0 + 0xdd6a (0x406d7d6a) 
libglib-1.2.so.0 + 0xf2c6 (0x406d92c6) 
libglib-1.2.so.0 + 0xf801 (0x406d9801) 
libglib-1.2.so.0 + 0xf979 (0x406d9979) 
libgtk-1.2.so.0 + 0x85f3a (0x4060bf3a) 
nsAppShell::Run() 
nsAppShellService::Run() 
main1() 
main() 
libc.so.6 + 0x17cb3 (0x401e4cb3)

Comment 1

[Jimmy Lee]

Correcting Platform/OS to Linux only.  Nominating for Beta.  This used to work.  
This is functionality we document as being supported.  This appears to work fine 
for Windows and Macintosh.

Comment 2

[Jimmy Lee]

I'm out of it.  Changing Severity to critical.  Adding keyword crash.

Comment 3

[Jimmy Lee]

Build: 2001-05-02-09-trunk(MAC)

This is reproducible on the Mac as well.  For some reason, it did work 
yesterday.  But I can "make" it crash now.

TalkBack Incident ID 29925617
Stack Trace
js_MarkGCThing() [jsgc.c, line 792] 
js_MarkGCThing() [jsgc.c, line 877] 
js_MarkGCThing() [jsgc.c, line 877] 
js_MarkGCThing() [jsgc.c, line 877] 
js_GC() [jsgc.c, line 1140] 
js_ForceGC() [jsgc.c, line 942] 
js_DestroyContext() [jscntxt.c, line 227] 
JS_DestroyContext() [jsapi.c, line 877] 
nsJSContext::~nsJSContext() [nsJSEnvironment.cpp, line 392] 
nsJSContext::Release() [nsJSEnvironment.cpp, line 400] 
nsCOMPtr_base::~nsCOMPtr_base() [nsCOMPtr.cpp, line 49] 
nsJSContext::CallEventHandler() [nsJSEnvironment.cpp, line 951] 
nsJSEventListener::HandleEvent() [nsJSEventListener.cpp, line 148] 
CONTENT_DLL + 0x86530 (0x1e4b3c80) 
CONTENT_DLL + 0x88674 (0x1e4b5dc4) 

Comment 4

[Jimmy Lee]

Attachment: Macintosh StdLog from Macsbug

Comment 5

[Jimmy Lee]

Updating Summary to indicate both Mac and Linux.

Comment 6

[Syd Logan]

please retest

Comment 7

[Jimmy Lee]

Build: 2001-05-09-08-trunk(MAC), 2001-05-08-08-trunk(LINUX)

The crashes are still occuring on both Mac and Linux.  No crash observed for 
Windows.

Comment 8

[Jimmy Lee]

Build: 2001-06-05-03-0.9.1(MAC), 2001-06-05-04-0.9.1(LINUX)

The crashes are no longer occuring on both Mac and Linux.  No crash observed for 
Windows.

Comment 9

[Syd Logan]

appears to be fixed

Comment 10

[Jimmy Lee]

Build: 2001-06-06-06-0.9.1(WIN), 2001-06-06-03-0.9.1(MAC), 
2001-06-06-04-0.9.1(LINUX)

Looks good on the branch.  Yessssssss!

Adding keyword vtrunk.

Comment 11

[Grace Bush]

verified on both trunk 2001062508 and branch 2001062508