67816 - javascript strict warnings in mailWindow.js

Status: VERIFIED INVALID

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Henrik Gemal]

JavaScript strict warning:
chrome://messenger/content/mailWindow.js line 323: reference to undefined 
property window.MsgStatusFeedback

JavaScript error:
chrome://messenger/content/mailWindow.js line 323: window.MsgStatusFeedback has
no properties

Comment 1

[[e-mail] (gone - use [e-mail] instead)]

Pardon my JS newbie questions here.  We know that window is an object of the
DOM, so is MsgStatusFeedback just another object of that, and _stopMeteors() is
just a property?  If so, would just declaring/setting the property
_stopMeteors() to be null or 0 before we reference it fix the warnings?  

window.MsgStatusFeedback._stopMeteors()

Comment 2

[[e-mail] (gone - use [e-mail] instead)]

<sigh> I believe I was off my rocket when I wrote that last comment.
_stopMeteors() has nothing to do with it.  Sorry </sigh> I'll just wait and see
how you fix it, Mark ;-) (Or whoever ends up fixing)

Comment 3

[Jason Johnston]

This is a pretty old bug that has dropped through the cracks-- it looks like
it's been resolved in the normal UI work since I don't see any JS errors using
mailnews (build 2001062704).  Of course the bug wasn't that descriptive to begin
with; steps to reproduce would have been nice.

Most likely can be closed.

Comment 4

[Alex Vincent [:WeirdAl]]

Given the lack of activity on this bug, the inability to reproduce in recent
builds, and the last comment six months ago recommending closure, I am marking
this bug INVALID.

Comment 5

[Alex Vincent [:WeirdAl]]

I oughta cc myself on any bug I mark invalid.  :)

Reporter:  If you can come up with a reproducible testcase, please feel free to
REOPEN this bug.

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

verifying this bug...

also note that my http://bugzilla.mozilla.org/show_bug.cgi?id=67816#c1 comment
was totally bogus (since, back then I didn't understand JS methods and functions
properly).