Closed Bug 1220716 Opened 4 years ago Closed 4 years ago

Remove nsICurrentCharsetListener

Categories

(Core :: Internationalization, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: hsivonen, Assigned: spillmas)

Details

(Whiteboard: [good first bug][lang=c++])

Attachments

(2 files)

nsICurrentCharsetListener is dead code. We should remove it.
Hello!

Could I have this bug assigned to me?

Thanks!
Attached patch Bug1220716.diffSplinter Review
All instances have been removed.
Attached patch Revision 1Splinter Review
Hi there.

It didn't look like Roland's comment3 quite removed everything. I've removed the code requested, built and run the tests with all passing. Seems to be all good to me.
Attachment #8697421 - Flags: review?(VYV03354)
Comment on attachment 8697421 [details] [diff] [review]
Revision 1

Looks good. Thanks for the contribution!
Attachment #8697421 - Flags: review?(VYV03354) → review+
Assignee: nobody → spillmas
https://hg.mozilla.org/mozilla-central/rev/226e81b0312f
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.