Closed Bug 44611 Opened 24 years ago Closed 24 years ago

Search messages: reset while search in progress can crash.

Categories

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

Tracking

(Not tracked)

VERIFIED WORKSFORME
Future

People

(Reporter: laurel, Assigned: gayatrib)

Details

(Keywords: crash, Whiteboard: [nsbeta3-])

Attachments

(1 file)

Using jul5 commercial build NT4.0 and linux rh6.0 Haven't gotten to mac yet, assume same results. Clicking the Reset button in Search Messages while a search is in progress crashes quite often. It may be a timing thing, it happens fairly easily on my slower machine and it's harder to hit on my faster ones doing the same search. Or it may involve clicking Reset a second time just at the right moment. Talkback reports submitted, will paste in comments when I get the reports. 1. Go to mail window, login to IMAP account. Select a folder with at least a few hundred messages (since we don't yet have account-wide search or search subfolders working, we need to get a search initiated which will take as much time as possible). 2. Search|Search Mail. Initiate a search such as: body, doesn't contain, o Note: I also tried on other criteria, but the body one seems to be easier to reproduce this with. 3. While the search is in progress click reset button. If it doesn't do anything, click reset again. Result: crashes in most cases, particularly on a slower machine.
Keywords: crash
QA Contact: lchiang → laurel
Talkback incident 13667946: Call Stack: (Signature = 0x00000000 72c190a5) 0x00000000 FindNextSibling [d:\builds\seamonkey\mozilla\layout\html\style\src\nsCSSFrameConstructor.cpp, line 8010] nsCSSFrameConstructor::ContentInserted [d:\builds\seamonkey\mozilla\layout\html\style\src\nsCSSFrameConstructor.cpp, line 8542] nsCSSFrameConstructor::RecreateFramesForContent [d:\builds\seamonkey\mozilla\layout\html\style\src\nsCSSFrameConstructor.cpp, line 10940] nsCSSFrameConstructor::AttributeChanged [d:\builds\seamonkey\mozilla\layout\html\style\src\nsCSSFrameConstructor.cpp, line 9979] StyleSetImpl::AttributeChanged [d:\builds\seamonkey\mozilla\layout\base\src\nsStyleSet.cpp, line 1143] PresShell::AttributeChanged [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 3296] nsXULDocument::AttributeChanged [d:\builds\seamonkey\mozilla\rdf\content\src\nsXULDocument.cpp, line 1674] nsXULElement::SetAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsXULElement.cpp, line 2849] nsXULElement::SetAttribute [d:\builds\seamonkey\mozilla\rdf\content\src\nsXULElement.cpp, line 1246] nsXULMenuListElement::SetSelectedItem [d:\builds\seamonkey\mozilla\rdf\content\src\nsXULMenuListElement.cpp, line 275] SetXULMenuListElementProperty [d:\builds\seamonkey\mozilla\rdf\content\src\nsJSXULMenuListElement.cpp, line 235] js_SetProperty [d:\builds\seamonkey\mozilla\js\src\jsobj.c, line 2254] js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2402] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 733] js_InternalInvoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 806] js_SetProperty [d:\builds\seamonkey\mozilla\js\src\jsobj.c, line 2151] js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2402] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 733] js_InternalInvoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 806] js_SetProperty [d:\builds\seamonkey\mozilla\js\src\jsobj.c, line 2151] js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2402] js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 733] js_InternalInvoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 806] JS_CallFunctionValue [d:\builds\seamonkey\mozilla\js\src\jsapi.c, line 2817] nsJSContext::CallEventHandler [d:\builds\seamonkey\mozilla\dom\src\base\nsJSEnvironment.cpp, line 850] nsJSEventListener::HandleEvent [d:\builds\seamonkey\mozilla\dom\src\events\nsJSEventListener.cpp, line 155] nsEventListenerManager::HandleEventSubType [d:\builds\seamonkey\mozilla\layout\events\src\nsEventListenerManager.cpp, line 773] nsEventListenerManager::HandleEvent [d:\builds\seamonkey\mozilla\layout\events\src\nsEventListenerManager.cpp, line 1653] nsXULElement::HandleDOMEvent [d:\builds\seamonkey\mozilla\rdf\content\src\nsXULElement.cpp, line 3350] PresShell::HandleDOMEventWithTarget [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 3956] nsTitledButtonFrame::MouseClicked [d:\builds\seamonkey\mozilla\layout\xul\base\src\nsTitledButtonFrame.cpp, line 1279] nsButtonBoxFrame::HandleEvent [d:\builds\seamonkey\mozilla\layout\xul\base\src\nsButtonBoxFrame.cpp, line 98] PresShell::HandleEventInternal [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 3925] PresShell::HandleEventWithTarget [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 3888] nsEventStateManager::CheckForAndDispatchClick [d:\builds\seamonkey\mozilla\layout\events\src\nsEventStateManager.cpp, line 1775] nsEventStateManager::PostHandleEvent [d:\builds\seamonkey\mozilla\layout\events\src\nsEventStateManager.cpp, line 854] PresShell::HandleEventInternal [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 3929] PresShell::HandleEvent [d:\builds\seamonkey\mozilla\layout\html\base\src\nsPresShell.cpp, line 3843] nsView::HandleEvent [d:\builds\seamonkey\mozilla\view\src\nsView.cpp, line 782] nsViewManager2::DispatchEvent [d:\builds\seamonkey\mozilla\view\src\nsViewManager2.cpp, line 1389] HandleEvent [d:\builds\seamonkey\mozilla\view\src\nsView.cpp, line 69] nsWindow::DispatchEvent [d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 564] nsWindow::DispatchWindowEvent [d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 581] nsWindow::DispatchMouseEvent [d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 3681] ChildWindow::DispatchMouseEvent [d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 3886] nsWindow::ProcessMessage [d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 2801] nsWindow::WindowProc [d:\builds\seamonkey\mozilla\widget\src\windows\nsWindow.cpp, line 830] USER32.dll + 0x1820 (0x77e71820)
ugh, this is a SCARY crash.
Attached file another talkback report 13668450 (deleted) —
should we disable reset when a search is going on? M18 and nsbeta3
Keywords: nsbeta3
Target Milestone: --- → M18
Whiteboard: [nsbeta3-]
Target Milestone: M18 → Future
If this is futured, does this mean we're also futuring the disabling of reset button when a search is in progress? This crash is pretty easy to hit if you're impatient and realize you specified the wrong criteria...
I'll disable the reset button
Status: NEW → ASSIGNED
add sspitzer to cc
adding waterson to the cc list. chris, this stack looks a lot like the one in bug #51571.
filters/search UI->gayatrib
Assignee: alecf → gayatrib
Status: ASSIGNED → NEW
Is this still happening?
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → WORKSFORME
No. Can't currently reproduce.
verified worksforme
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
Component: MailNews: Search → MailNews: Message Display
QA Contact: laurel → search
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: