63921 - Rename/Deleted IMAP folders do not refresh on folder pane & trash folder until expand/collapse folders or relaunch app

Status: VERIFIED FIXED

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

Description

[Sheela Ravindran]

buildid: 2000122908
os: mac
Steps to reproduce
Open mail and select any existing folder and click rename
Rename folder dlg appears
Rename an 'xyz'existing folder to 'abc'
click ok
And do not click any where on that screen .  Just wait
You will notice that the folder does not refresh on its own
Workaround:  Click on the folder you just renamed and will refresh.

Comment 1

[Sheela Ravindran]

change qa contact

Comment 2

[Sheela Ravindran]

*** Bug 64143 has been marked as a duplicate of this bug. ***

Comment 3

[Karen Huang]

Based on 64143 is nsbeta1+ bug, adding mail1 & nsbeta1 for the keywords to 
ensure this bug can be fixed for next release.

Comment 4

[Navin Gupta]

reassigning to self because a dup has been assigned to me.

Comment 5

[Navin Gupta]

really reassigning. Sorry for extra mail.

Comment 6

[scottputterman]

marking nsbeta1+ and moving to mozilla0.8

Comment 7

[Fabian Guisset]

if by refresh you mean that the name in the folder pane is not updated..
worksforme build 2001010320 win95.

Comment 8

[Sheela Ravindran]

sorry for not replying late. Yes, I did mean that the folder pane does not get
updated. This has been seen on winNT too.( bug 64143)

Comment 9

[Karen Huang]

*** Bug 64443 has been marked as a duplicate of this bug. ***

Comment 10

[Karen Huang]

Changing summary to ensure for covering Rename & Delete folders scenarios, 
Adding IMAP on the summary too.
And if this is occurring on IMAP, then QA should assign to me.
Change QA to me and Ccing Sheela.

Comment 11

[Karen Huang]

*** Bug 65147 has been marked as a duplicate of this bug. ***

Comment 12

[scottputterman]

moving to mozilla0.9

Comment 13

[scottputterman]

marking nsbeta1- and moving to future milestone.

Comment 14

[Karen Huang]

Renominating this bug for nsbeta1.
Without this fix, users will misunderstand that the rename folder functionality 
is not working! Adding pp for the keywords since problem is ONLY occurs on WinNT 
& Mac platforms.

Comment 15

[Navin Gupta]

Yes I also want this to be nominated 

Comment 16

[Navin Gupta]

This assertions causes rename to fail on opt build but you can ignore it 
on the debug build so it work fine.

NTDLL! 77f7629c()
nsDebug::Assertion(const char * 0x100e4d38, const char * 0x100e4d2c, const char 
* 0x100e4cfc, int 238) line 254 + 13 bytes
nsEventQueueImpl::PostEvent(nsEventQueueImpl * const 0x03f05e50, PLEvent * 
0x05a07c34) line 238 + 40 bytes
nsStreamObserverEvent::FireEvent(nsIEventQueue * 0x03f05e50) line 65 + 19 bytes
nsStreamListenerProxy::OnDataAvailable(nsStreamListenerProxy * const 0x03f01824, 
nsIChannel * 0x03b3bd64, nsISupports * 0x00000000, nsIInputStream * 0x05978dd0, 
unsigned int 3694, unsigned int 8192) line 315 + 20 bytes
nsSocketTransport::doReadAsync(short 1) line 1290 + 96 bytes
nsSocketTransport::Process(short 1) line 808 + 13 bytes
nsSocketTransportService::Run(nsSocketTransportService * const 0x017dbcc4) line 
424 + 13 bytes
nsThread::Main(void * 0x017db720) line 106 + 26 bytes
_PR_NativeRunThread(void * 0x017db500) line 399 + 13 bytes
_threadstartex(void * 0x017db350) line 212 + 13 bytes
KERNEL32! 77f04ee8()

I will wait for dougt necko landing to see if that fixes this problem

Comment 17

[scottputterman]

remarking nsbeta1-.  It sounds like this is causing a ui problem that fixes 
itself relatively easily.

Comment 18

[Karen Huang]

Removing nsbeta1-, renominating nsbeta1 -- I really think this should be fix in 
next release.....

Comment 19

[Navin Gupta]

This bug blocks 50722 & 64812 and we need to fix this. I think it has 
definitely to do with the above assertion. 

cc' ing mscott for his opinion

Comment 20

[Karen Huang]

*** Bug 68154 has been marked as a duplicate of this bug. ***

Comment 21

[scottputterman]

I'd much rather see the 3 crashes fixed then this bug.

Comment 22

[scottputterman]

marking nsbeta1+ and moving to mozilla0.9

Comment 23

[Navin Gupta]

cc dougt, Do you know anything about the above assertion ?

Comment 24

[Doug Turner (:dougt)]

NS_ASSERTION(mElderQueue, "event dropped because event chain is dead");


darin, have you seen this when playing around in mail after your socket 
changes?

Comment 25

[Navin Gupta]

Attachment: patch to get rid of assertion and a possible fix

Comment 26

[David :Bienvenu]

sr=bienvenu

Comment 27

[Navin Gupta]

fix checked in last night. Thanks to bienvenu for helping me.

Comment 28

[Karen Huang]

Verified on Mac 05-02-09-trunk & Win 05-02-12-trunk builds.