309061 - msgHdrViewOverlay.js should use hasChildNodes

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Ian Neal]

Once the fix to bug 240633 landed one of the changes made to TB was
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=msgHdrViewOverlay.js&branch=&root=/cvsroot&subdir=mozilla/mail/base/content&command=DIFF_FRAMESET&rev1=1.18&rev2=1.19

This is the equivalent bug for mailnews

Comment 1

[Ian Neal]

Attachment: Use hasChildNodes patch v0.1 (Checked in trunk/branch 1.8 & 1.8.0)

This patch:
* Changes to using hasChildNodes()

Comment 2

[[e-mail]]

Comment on attachment 196568 [details] [diff] [review]
Use hasChildNodes patch v0.1 (Checked in trunk/branch 1.8 & 1.8.0)

r+sr=me

Comment 3

[Ian Neal]

Comment on attachment 196568 [details] [diff] [review]
Use hasChildNodes patch v0.1 (Checked in trunk/branch 1.8 & 1.8.0)

Checking in
mozilla/mailnews/base/resources/content/msgHdrViewOverlay.js;
new revision: 1.143; previous revision: 1.142
done

Comment 4

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Comment on attachment 196568 [details] [diff] [review]
Use hasChildNodes patch v0.1 (Checked in trunk/branch 1.8 & 1.8.0)

first a=me

Comment 5

[Ian Neal]

Comment on attachment 196568 [details] [diff] [review]
Use hasChildNodes patch v0.1 (Checked in trunk/branch 1.8 & 1.8.0)

Checking in (branch)
msgHdrViewOverlay.js;
new revision: 1.141.2.3; previous revision: 1.141.2.2
done

Comment 6

[Ian Neal]

Comment on attachment 196568 [details] [diff] [review]
Use hasChildNodes patch v0.1 (Checked in trunk/branch 1.8 & 1.8.0)

Checking in (branch 1.8.0)
msgHdrViewOverlay.js;
new revision: 1.141.2.1.2.2; previous revision: 1.141.2.1.2.1
done

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED using build 2006-01-06-09 of trunk SeaMonkey under Windows XP.

No apparent regressions, and the feature works...