Well, here's an unsurprising bug... when you remove ("Cut") the bookmarks-folder from the toolbars, and subsequently try to create another bookmark, things crash since the bookmark-folder doesn't exist. Presumably, Mozilla shouldn't allow one to Cut off the active bookmark-folder (Like it doesn't allow you to do with new bookmarks-folder).

Also crashes under a number of other combinations: Mark folder for both bookmark menu & new bookmarks, for example. I'm in a hurry and no time to analyze these different conditions, just adding a note there seems to be a lot wrong with the new custom toolbars and bookmarks ;)

Can't reproduce this problem with the latest version off CVS anymore; It's still possible to delete (Cut) an active bookmark-folder, but not the new bookmark folder. I don't know if it's actually been fixed, so changing WORKSFORME. If somebody else can reproduce it, or knows it's been patched, mark it...

Trust me to log resolved just before crashing it again. I'm not sure if this is the same problem, but it looks certainly related, so I'm reopening. After for a while deleting and adding bookmarks thru the menubar, I decided to check the next bug, clicked on the custom taskbar and selected "Add bookmark...", in // check to make sure the VTable pointer is valid __vfptr is 0xdddddddd AfxAssertValidObject(const CObject * 0x01c42cf0 {CObject}, char * 0x5f4c0d78, int 2214) line 96 CWnd::GetTopLevelFrame() line 2216 CRDFCX::GetDialogOwner() line 4201 CStubsCX::Prompt(MWContext_ * 0x00a3d980, char * 0x00b13350, char * 0x00b13380) line 434 + 20 bytes CFE_Prompt(MWContext_ * 0x00a3d980, char * 0x008cd80c, char * 0x008cd804) line 1036 + 32 bytes HT_DoMenuCmd(_HT_PaneStruct * 0x00a5ada0, int 7) line 4304 + 40 bytes CRDFToolbar::OnCommand(unsigned int 16601, long 0) line 2410 + 28 bytes CWnd::OnWndMsg(unsigned int 273, unsigned int 16601, long 0, long * 0x0012fdbc) line 1587 + 28 bytes CWnd::WindowProc(unsigned int 273, unsigned int 16601, long 0) line 1575 + 30 bytes AfxCallWndProc(CWnd * 0x00a96120 {CRDFToolbar hWnd=0x00020570}, HWND__ * 0x00020570, unsigned int 273, unsigned int 16601, long 0) line 217 + 26 bytes AfxWndProc(HWND__ * 0x00020570, unsigned int 273, unsigned int 16601, long 0) line 371 AfxWndProcBase(HWND__ * 0x00020570, unsigned int 273, unsigned int 16601, long 0) line 203 + 21 bytes USER32! 77e713ed()

Thanks, this at least lets me know who to hand off to.

Marking Verified/Invalid.