24288 - editing account name should reflect in tree view

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect, P3)

Description

[Warren Harris]

I brought up the Account Setup dialog and created a new POP profile. Then I
selected the POP profile and edited its name. After going to another field in
the dialog, I noticed that the name didn't get updated in the left-hand column
(the tree-view widget). Only after closing and reopening the Account Setup
dialog did I see the new name.

On a related note, the second time I tried to open the account setup dialog, the
whole mail/news app hung. I had to quit the browser and restart to see the
edited name. Before I quit though, I broke in to see what was going on and it
seemed to be hung in some dialog code. I didn't get a stack trace.

Comment 1

[Warren Harris]

Seems reproducible. Here's the stacktrace of the hang. Looks like it's stuck in
GetMessage, waiting for an event:

USER32! 77e724b2()
nsAppShell::GetNativeEvent(nsAppShell * const 0x0321dcf0, int & 0x00000001,
void * & 0x01420398 msg) line 123 + 17 bytes
nsWebShellWindow::ShowModalInternal(nsWebShellWindow * const 0x031ab820) line
1782 + 20 bytes
nsWebShellWindow::ShowModal(nsWebShellWindow * const 0x031ab820) line 1748 + 9
bytes
nsWebShellWindow::ShowModally(nsWebShellWindow * const 0x031ab828, int
0x00000000) line 1824 + 16 bytes
GlobalWindowImpl::OpenInternal(JSContext * 0x02864850, long * 0x02272084,
unsigned int 0x00000004, int 0x00000001, nsIDOMWindow * * 0x0012dfec) line 2514
GlobalWindowImpl::OpenDialog(GlobalWindowImpl * const 0x028649e8, JSContext *
0x02864850, long * 0x02272084, unsigned int 0x00000004, nsIDOMWindow * *
0x0012dfec) line 2355
WindowOpenDialog(JSContext * 0x02864850, JSObject * 0x02229690, unsigned int
0x00000004, long * 0x02272084, long * 0x0012e0a8) line 2607 + 31 bytes
js_Invoke(JSContext * 0x02864850, unsigned int 0x00000004, unsigned int
0x00000000) line 665 + 26 bytes
js_Interpret(JSContext * 0x02864850, long * 0x0012e918) line 2226 + 15 bytes
js_Invoke(JSContext * 0x02864850, unsigned int 0x00000000, unsigned int
0x00000000) line 681 + 13 bytes
js_Interpret(JSContext * 0x02864850, long * 0x0012f144) line 2226 + 15 bytes
js_Invoke(JSContext * 0x02864850, unsigned int 0x00000001, unsigned int
0x00000002) line 681 + 13 bytes
js_InternalCall(JSContext * 0x02864850, JSObject * 0x022dcf88, long 0x022dcf90,
unsigned int 0x00000001, long * 0x0012f2c8, long * 0x0012f274) line 758 + 15
bytes
JS_CallFunctionValue(JSContext * 0x02864850, JSObject * 0x022dcf88, long
0x022dcf90, unsigned int 0x00000001, long * 0x0012f2c8, long * 0x0012f274) line
2758 + 29 bytes
nsJSContext::CallEventHandler(nsJSContext * const 0x028614f0, void *
0x022dcf88, void * 0x022dcf90, unsigned int 0x00000001, void * 0x0012f2c8, int
* 0x0012f2c4) line 564 + 33 bytes
nsJSEventListener::HandleEvent(nsIDOMEvent * 0x031af884) line 128 + 57 bytes
nsEventListenerManager::HandleEventSubType(nsListenerStruct * 0x02cfe890,
nsIDOMEvent * 0x031af884, unsigned int 0x00000008) line 640 + 19 bytes
nsEventListenerManager::HandleEvent(nsIPresContext * 0x028a9b80, nsEvent *
0x0012f7f0, nsIDOMEvent * * 0x0012f7bc, unsigned int 0x00000007, nsEventStatus
* 0x0012f834) line 1399 + 31 bytes
nsXULElement::HandleDOMEvent(nsXULElement * const 0x02cfe9c0, nsIPresContext *
0x028a9b80, nsEvent * 0x0012f7f0, nsIDOMEvent * * 0x0012f7bc, unsigned int
0x00000001, nsEventStatus * 0x0012f834) line 2795
nsMenuFrame::Execute() line 1244
nsMenuFrame::HandleEvent(nsMenuFrame * const 0x022fa824, nsIPresContext *
0x028a9b80, nsGUIEvent * 0x0012fb6c, nsEventStatus * 0x0012fa78) line 286
PresShell::HandleEvent(PresShell * const 0x028a9194, nsIView * 0x03012700,
nsGUIEvent * 0x0012fb6c, nsEventStatus * 0x0012fa78) line 2759 + 38 bytes
nsView::HandleEvent(nsView * const 0x03012700, nsGUIEvent * 0x0012fb6c,
unsigned int 0x00000008, nsEventStatus * 0x0012fa78, int & 0x00000000) line 841
nsView::HandleEvent(nsView * const 0x028a9690, nsGUIEvent * 0x0012fb6c,
unsigned int 0x0000001c, nsEventStatus * 0x0012fa78, int & 0x00000000) line 826
nsViewManager2::DispatchEvent(nsViewManager2 * const 0x028a98b0, nsGUIEvent *
0x0012fb6c, nsEventStatus * 0x0012fa78) line 1002
HandleEvent(nsGUIEvent * 0x0012fb6c) line 69
nsWindow::DispatchEvent(nsWindow * const 0x030125d4, nsGUIEvent * 0x0012fb6c,
nsEventStatus & nsEventStatus_eIgnore) line 502 + 10 bytes
nsWindow::DispatchWindowEvent(nsGUIEvent * 0x0012fb6c) line 523
nsWindow::DispatchMouseEvent(unsigned int 0x0000012d, nsPoint * 0x00000000
{x=??? y=???}) line 3465 + 21 bytes
ChildWindow::DispatchMouseEvent(unsigned int 0x0000012d, nsPoint * 0x00000000
{x=??? y=???}) line 3683
nsWindow::ProcessMessage(unsigned int 0x00000202, unsigned int 0x00000000, long
0x01140033, long * 0x0012fdcc) line 2755 + 24 bytes
nsWindow::WindowProc(HWND__ * 0x1646125a, unsigned int 0x00000202, unsigned int
0x00000000, long 0x01140033) line 689 + 27 bytes
USER32! 77e71820(

Comment 2

[Alec Flett]

that stack trace should go in a new bug, it's unrelated.

Comment 3

[Warren Harris]

Ok, keeping this bug for updating the tree view's account name. Opened bug
#24328 for the deadlock.

Comment 4

[laurel]

This was originally reported in #18987 which was marked duplicate of #17713

Comment 5

[Alec Flett]

Fixed! The account manager is now hooked up to RDF asynchronously!

Comment 6

[Ninoschka Baca]

Build 2000-02-11-08M14: NT4
Build 2000-02-11-11M14: Linux 6.0
Build 2000-02-11-12M14: Mac 8.6.1
Verified Fixed. Changes on any of the Account Setup panels are changed 
immediately.