Closed
Bug 1143110
Opened 10 years ago
Closed 10 years ago
remove unnecessary nsISSLErrorListener implementation in chatzilla
Categories
(Other Applications Graveyard :: ChatZilla, defect)
Other Applications Graveyard
ChatZilla
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: keeler, Assigned: keeler)
References
Details
(Whiteboard: [cz-0.9.92])
Attachments
(1 file)
|
1.80 KB,
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
nsISSLErrorListener is obsolete and will be removed in bug 844351. The implementation in chatzilla don't do anything and can be removed.
| Assignee | ||
Comment 1•10 years ago
|
||
Gijs, would you mind reviewing this? Thanks.
Assignee: rginda → dkeeler
Status: NEW → ASSIGNED
Attachment #8577424 -
Flags: review?(gijskruitbosch+bugs)
Comment 2•10 years ago
|
||
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+
| Assignee | ||
Comment 3•10 years ago
|
||
Thanks! What's the process for checking in to chatzilla? (should I just mark this checkin-needed?)
Comment 4•10 years ago
|
||
(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).
| Assignee | ||
Comment 5•10 years ago
|
||
Ok - pushed:
https://hg.mozilla.org/chatzilla/rev/f8e44816adce
Thanks!
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Whiteboard: [cz-0.9.92]
Updated•7 months ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•