Closed Bug 1143110 Opened 10 years ago Closed 10 years ago

remove unnecessary nsISSLErrorListener implementation in chatzilla

Categories

(Other Applications Graveyard :: ChatZilla, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: keeler, Assigned: keeler)

References

Details

(Whiteboard: [cz-0.9.92])

Attachments

(1 file)

nsISSLErrorListener is obsolete and will be removed in bug 844351. The implementation in chatzilla don't do anything and can be removed.
Attached patch patchSplinter Review
Gijs, would you mind reviewing this? Thanks.
Assignee: rginda → dkeeler
Status: NEW → ASSIGNED
Attachment #8577424 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8577424 [details] [diff] [review] patch Review of attachment 8577424 [details] [diff] [review]: ----------------------------------------------------------------- r+ because this was apparently non-functional since at least December 2011: http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=7cd14fbf2789 which is a few ESRs ago
Attachment #8577424 - Flags: review?(gijskruitbosch+bugs) → review+
Thanks! What's the process for checking in to chatzilla? (should I just mark this checkin-needed?)
(In reply to David Keeler [:keeler] (use needinfo?) from comment #3) > Thanks! What's the process for checking in to chatzilla? (should I just mark > this checkin-needed?) I believe that if you can push to m-c (integration), then you should be able to push to chatzilla's repo, too. If you prefer, I can also land this for you (but please say so).
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [cz-0.9.92]
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: