919935 - EncodingUtils should have a deCOMtaminated replacement for GetUnicodeDecoderRaw

Status: RESOLVED FIXED

(Core :: Internationalization, defect)

Description

[Henri Sivonen (:hsivonen) (away from Bugzilla until 2026-04-13)]

We could make code cleaner and less confusing if after EncodingUtils::FindEncodingForLabel we called something like
already_AddRefed<nsIUnicodeDecoder> EncodingUtils::DecoderForEncoding(const nsACString& aEncoding); instead of dealing with the nsICharsetConverterManager service.

Comment 1

[Henri Sivonen (:hsivonen) (away from Bugzilla until 2026-04-13)]

Attachment: Implement deCOMtaminated decoder/encoder factories

https://tbpl.mozilla.org/?tree=Try&rev=f0327233b42b

Comment 2

[Henri Sivonen (:hsivonen) (away from Bugzilla until 2026-04-13)]

Attachment: Add deCOMtaminated factories

Comment 3

[Masatoshi Kimura [:emk]]

Comment on attachment 8337699 [details] [diff] [review]
Add deCOMtaminated factories

Review of attachment 8337699 [details] [diff] [review]:
-----------------------------------------------------------------

::: intl/uconv/idl/nsICharsetConverterManager.idl
@@ +23,5 @@
>  interface nsIUTF8StringEnumerator;
>  
>  /**
> + * DON'T ADD NEW USES OF THIS INTERFACE TO MOZILLA-CENTRAL. Use 
> + * mozilla::dom::EncodingUtils instead.

@deprecated

::: parser/html/nsHtml5StreamParser.cpp
@@ +322,2 @@
>    mCharset.Assign(aDecoderCharsetName);
> +  mUnicodeDecoder = EncodingUtils::DecoderForEncoding(mCharset);

aDecoderCharsetName?

Comment 4

[Henri Sivonen (:hsivonen) (away from Bugzilla until 2026-04-13)]

(In reply to Masatoshi Kimura [:emk] from comment #3)
> Comment on attachment 8337699 [details] [diff] [review]
> Add deCOMtaminated factories
> 
> Review of attachment 8337699 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: intl/uconv/idl/nsICharsetConverterManager.idl
> @@ +23,5 @@
> >  interface nsIUTF8StringEnumerator;
> >  
> >  /**
> > + * DON'T ADD NEW USES OF THIS INTERFACE TO MOZILLA-CENTRAL. Use 
> > + * mozilla::dom::EncodingUtils instead.
> 
> @deprecated

Added.

> ::: parser/html/nsHtml5StreamParser.cpp
> @@ +322,2 @@
> >    mCharset.Assign(aDecoderCharsetName);
> > +  mUnicodeDecoder = EncodingUtils::DecoderForEncoding(mCharset);
> 
> aDecoderCharsetName?

I think passing mCharset as the argument is more idiomatic. Maybe I should have changed the name of aDecoderCharsetName in a previous bug, though.

Thanks.

Landed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/045001182507

Try run for sheriff reference:
https://tbpl.mozilla.org/?tree=Try&rev=4024f26b425b

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/045001182507