71468 - ChatZilla 0.8 superbug

Status: VERIFIED FIXED

(Other Applications :: ChatZilla, defect)

Description

[Robert Ginda]

This bug is indended to track the progress of reviews leading to the checkin of
ChatZilla version 0.8.  The XPI for 0.8 can be installed from the hacksrus link
listed in the URL field of this bug.

Comment 1

[Robert Ginda]

Attachment: ChangeLog. The changes listed for 0.8 are listed in the order they appear in the diff.

Comment 2

[Robert Ginda]

Attachment: diff of changes for 0.8

Comment 3

[Robert Ginda]

Attachment: outputwindow.html, new file for 0.8

Comment 4

[Robert Ginda]

looking for reviews from gary and ssieb, and an sr from scc.  I'd love to see
this get checked in by the 0.8.1 closure.

Comment 5

[Robert Ginda]

Updates to this patch so far...

typeo-s in changelog fixed (timeless)
s/var seconds/seconds/ in utils.js:formatDateOffset() (ssieb)
s/hours = hours % 60/hours = hours % 24/ in utils.js:formatDateOffset() (ssieb)
s/w.open(/w.openDialog(/ in chatzilla-service.js:handleContent() (rginda)

Comment 6

[Robert Ginda]

fix for |connected| not getting set to false on c.connect(...) failure
irc.js:onDoConnect (peterv)

Comment 7

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

sr=shaver.

Comment 8

[Samuel Sieb]

r=ssieb

Comment 9

[Robert Ginda]

fix checked in.

Comment 10

[Samuel Sieb]

verified

Comment 11

[Asa Dotzler [:asa]]

Verified on 03/14 mozilla win32, linux and mac builds :)  Woo-Hoo! in time for 0.8.1