132009 - Delete in standalone msg window doesn't advance to next message

Status: VERIFIED DUPLICATE of bug 131769

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

Description

[Mike Pinkerton (not reading bugmail)]

osx debug mozilla build from 3/18/02

-open message in standalone window
-click delete button

expected:
- msg deletes and advances to next unread message, or closes

actual:
- msg deletes.

this makes reading mail a pain in the butt.

Comment 1

[laurel]

Request for close standalone window on delete is bug 80574.
In many other bugs on this subject, navigation is discussed but has never been
decided upon.

Comment 2

[Mike Pinkerton (not reading bugmail)]

laurel, what you are missing is that this worked fine 2 days ago. this is a 
regression of perfectly functional code.

Comment 3

[laurel]

Right, sorry for the oversight. 

Comment 4

[(not reading, please use [e-mail] instead)]

accepting.

cc ssu and bienvenu, active hackers.

Comment 5

[(not reading, please use [e-mail] instead)]

I can reproduce this.  my gut (in its infinite wisdom, and size, as bienvenu 
likes to point out) says this regression is caused by ssu's recent changes.

Comment 6

[David :Bienvenu]

Although it certainly pays to listen to Seth's gut, I have a feeling that I may
have broken this - it works in my tree, which has a couple of fixes that I
haven't checked in yet.

Comment 7

[(not reading, please use [e-mail] instead)]

I'm seeing that in messageWindow.js, gCurrentMessageIsDeleted is false, in 
HandleDeleteOrMoveMsgCompleted()

Comment 8

[(not reading, please use [e-mail] instead)]

I'm debugging OnItemRemoved() in messageWindow.js now.

That's where gCurrentMessageIsDeleted should get set to true.

Comment 9

[(not reading, please use [e-mail] instead)]

it looks like OnItemRemoved() never gets called.

Comment 10

[(not reading, please use [e-mail] instead)]

my apologies to pointing the finger at ssu.  

It's really a dup of #131769, bienvenu has a fix.

I tested it, and the patch in that bug fixes this problem.

*** This bug has been marked as a duplicate of 131769 ***

Comment 11

[laurel]

verified OK mar 26 commercial trunk: win98, mac OS 9.2, linux rh6.2