7568 - [PP] Crash when selecting Acct Manager panels in succession.

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P1)

Description

[laurel]

Using 1999-06-03-08 m7 on Linux rh5.2
Using 1999-06-03-08 m7 on Mac OS 8.51
Haven't been able to reproduce using NT 4.0

A crash or freeze occurs when opening the Account Manager dialog, selecting and
expanding a server then selecting the server's panels in succession fairly
quickly.

1.  Launch Messenger.
2.  Select Inbox, wait for messages to load.
3.  Edit|Account Manager.
    Wait for dialog to draw (mac requires resizing the dialog before the dialog
fully displays).
4.  Select a server (didn't appear to matter whether multiple servers were
present or what type I selected) then double-click to expand so that the panel
choices Main, Server, Copies and Advanced appear beneath the server name.
5.  Select each of the panel choices in succession.
Result:  crashes or freezes application.

Can't get Mac Talkback, but here's a linux gdb trace:

(gdb) where
#0  0x8b54c1c in ?? ()
#1  0x40a8153c in CSSLoaderImpl::DidLoadStyle ()
#2  0x40a80e6e in CSSLoaderImpl::RecycleParser ()
#3  0x402449c2 in nsUnicharStreamLoader::OnStopBinding ()
#4  0x402597c4 in nsDocumentBindInfo::OnStopBinding ()
#5  0x40246d68 in XP_FindContextOfType ()
#6  0x4018abfb in _init ()
#7  0x40223f3f in NET_ProcessNet ()
#8  0x40229563 in NET_PollSockets ()
#9  0x4024229d in nsNetlibService::NetPollSocketsCallback ()
#10 0x400f727e in TimerImpl::FireTimeout ()
#11 0x400f75e0 in nsTimerExpired ()
#12 0x80e6b53 in g_main_iteration ()
#13 0x80e60d8 in g_list_length ()
---Type <return> to continue, or q <return> to quit---
#14 0x80e6553 in g_list_length ()
#15 0x80e65ca in g_main_iteration ()
#16 0x400b1377 in nsAppShell::GetNativeEvent ()
#17 0x4001c134 in nsWebShellWindow::ShowModalInternal ()
#18 0x4001c094 in nsWebShellWindow::ShowModal ()
#19 0x406e4c52 in nsMsgAppCore::AccountManager ()
#20 0x406e5e04 in nsMsgAppCore::AccountManager ()
#21 0x4035dd82 in js_Invoke ()
#22 0x403639d6 in js_Interpret ()
#23 0x4035ddd0 in js_Invoke ()
#24 0x403639d6 in js_Interpret ()
#25 0x4035ddd0 in js_Invoke ()
#26 0x4035df75 in js_CallFunctionValue ()
#27 0x40348c99 in JS_CallFunctionValue ()
#28 0x402f0ed0 in nsJSEventListener::HandleEvent ()
#29 0x409b03a9 in nsEventListenerManager::HandleEvent ()
#30 0x407d02c8 in RDFElementImpl::HandleDOMEvent ()
#31 0x400b8642 in nsMenuItem::DoCommand ()
#32 0x400b8479 in nsMenuItem::MenuItemSelected ()
#33 0x400b918a in menu_item_activate_handler ()
#34 0x80c395a in gtk_window_set_default_size ()
#35 0x809c59b in gtk_signal_connect_object ()
#36 0x809bc22 in gtk_signal_connect_object ()
#37 0x809a35e in gtk_selection_data_set ()
#38 0x80bbfac in gtk_widget_size_request ()
#39 0x808b119 in gtk_menu_shell_append ()
#40 0x808a61a in gtk_menu_shell_append ()
#41 0x80c3740 in gtk_window_set_default_size ()
#42 0x809bc4f in gtk_signal_connect_object ()
#43 0x809a35e in gtk_selection_data_set ()
#44 0x80bbe88 in gtk_widget_size_request ()
#45 0x80854dd in gtk_get_current_event ()
#46 0x8084a7e in gtk_main_iteration_do ()
#47 0x80d3153 in gdk_input_add ()
#48 0x80e60d8 in g_list_length ()
#49 0x80e6553 in g_list_length ()
#50 0x80e666d in g_main_iteration ()
#51 0x8084593 in gtk_main ()
#52 0x400b12c3 in nsAppShell::Run ()
#53 0x40018fb6 in nsAppShellService::Run ()
#54 0x8051327 in main ()
(gdb)

Comment 1

[Alec Flett]

eeugh. I'm pushing this to M-something in the hopes that it gets fixed before I
get to it (I think it's an XPFE or CSS bug, since the whole account manager is
written in JS)

Anyway, the account manager will eventually be merged into the prefs anyway.

Comment 2

[lchiang]

Will add to release notes for M7 since the result is a crash.

Comment 3

[Alec Flett]

I think this is the same as bug 7902

Comment 4

[lchiang]

Ninoschka - can you see if this crash is happening in more recent builds?
Thanks.

Comment 5

[Ninoschka Baca]

alec - It no longer crashes in the latest builds (1999080408M9: Win32/NT4,
Linux/Redhat 5.2, Mac 8.5.1). Should I wait until bug# 11139 is fixed, where the
panels will change, to try it again?

Comment 6

[Alec Flett]

yes, thanks

Comment 7

[Ninoschka Baca]

Changing QA Contact from laurel to nbaca.

Comment 8

[sol]

Sounds like a bug that should be fixed for PR1 (if it's not already fixed), so I
added a note to the Status Whiteboard

Comment 9

[Ninoschka Baca]

Build 1999081608M9:

- On Mac 8.5.1 it freezes the application when selecting panels in succession.
Using 5 accounts or just 1 account produce the same results.
- On Win32/NT4 it appears fixed.
- On Linux/Redhat 6.0 it appears fixed.

Comment 10

[Alec Flett]

it's looking like this is possibly related to a leak in the style system.
I may end up reassigning this to the layout group

Comment 11

[Alec Flett]

ok, this has been fixed.
I think we leak alot to the X server when we load content into IFRAMEs.
I've worked around this by loading 3 IFRAMEs for the three pages I need to load,
and then hiding them inside a <deck> and only showing the pages I need on demand

Comment 12

[Ninoschka Baca]

Build 1999083108M10: Mac 8.5.1, Linux/Redhat 6.0 and 1999090109M10: Win32/NT4
Verified Fixed.
I used a pref with one account and 6 accounts, clicking on various Account Setup
panels without a crash.