459045 - very first notification is empty

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[u294409]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; pl; rv:1.9.0.3) Gecko/2008092416 openSUSE/11.0 Firefox/3.0.3
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; pl; rv:1.9.1b1pre) Gecko/20081008 Lightning/1.0pre Shredder/3.0b1pre

every first notification in thunderbird is empty. every next isn't.

Reproducible: Always

Steps to Reproduce:
1. download TB nightly
2. configure
3. close and open again
Actual Results:  
empty notification

Expected Results:  
filled notification

Comment 1

[u294409]

Attachment: very first notification

Comment 2

[David :Bienvenu]

I'm seeing this as well. I believe it's because we have a folder in the list of folders with new messages that doesn't have new messages, and it makes the newmailalert code bail out early. I'll try to debug this when I get a chance...

Comment 3

[Jesse Ruderman on Windows]

Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.4pre) Gecko/20090914 Shredder/3.0b4pre

This happens every time I start Thunderbird and have new mail, not only the first or second time I use Thunderbird.

Comment 4

[David Ascher (:davida)]

not just linux i guess.

Comment 5

[David Ascher (:davida)]

QA, it'd be super if we could get reliable STRs for this one.

Comment 6

[David :Bienvenu]

Attachment: get rid of empty alert

this gets rid of the empty alert - in fact, it gets rid of the alert completely in the error case. I'd like to know why we're getting into this situation, but this fix should do for now.

Comment 7

[Ludovic Hirlimann [:Usul]]

Attachment: Folder for said account

(In reply to comment #5)
> QA, it'd be super if we could get reliable STRs for this one.

I have a gmail account I'm willing to share that exhibit this behavior on first install. And does it reliabely.

Bienvenu see the screenshot for folders and their state.

Comment 8

[Magnus Melin [:mkmelin]]

Comment on attachment 402724 [details] [diff] [review]
get rid of empty alert

The return makes us not prefill the alert, but does it really not show at all?

Bug 471772 is likely dupe (cause wise)

Comment 9

[David :Bienvenu]

(In reply to comment #8)
> (From update of attachment 402724 [details] [diff] [review])
> The return makes us not prefill the alert, but does it really not show at all?

Seems to, in my tests - basically, we don't throw an exception with this patch, which seems to make things a lot cleaner.

> 
> Bug 471772 is likely dupe (cause wise)

Comment 10

[Magnus Melin [:mkmelin]]

Comment on attachment 402724 [details] [diff] [review]
get rid of empty alert

r=mkmelin

That 160 char row could be wrapped though.

Comment 11

[David :Bienvenu]

fix checked in, with long line wrapped