UTF8InputStream is redundant with nsConverterInputStream initialized with UTF-8 and uses facilities that I want to remove as part of bug 1402247.

Comment on attachment 8958932 [details] Bug 1445692 - Remove UTF8InputStream. https://reviewboard.mozilla.org/r/227796/#review235092 ::: xpcom/io/nsUnicharInputStream.cpp:139 (Diff revision 3) > - RefPtr<UTF8InputStream> it = new UTF8InputStream(); > - nsresult rv = it->Init(aStreamToWrap); > + RefPtr<nsConverterInputStream> it = new nsConverterInputStream(); > + nsresult rv = > + it->Init(aStreamToWrap, > + "UTF-8", > + STRING_BUFFER_SIZE, > + 0); Would it be worth adding a comment here as to the meaning of `0`? Or, alternatively, WDYT about adding: ``` const char16_t THROW_EXCEPTIONS_ON_INVALID_BYTE_SEQUENCES = 0; ``` to `nsIConverterInputStream.idl` (name subject to bikeshedding), so we don't have these weird constants hanging out in `Init` calls?

Comment on attachment 8958932 [details] Bug 1445692 - Remove UTF8InputStream. https://reviewboard.mozilla.org/r/227796/#review235092 > Would it be worth adding a comment here as to the meaning of `0`? Or, alternatively, WDYT about adding: > > ``` > const char16_t THROW_EXCEPTIONS_ON_INVALID_BYTE_SEQUENCES = 0; > ``` > > to `nsIConverterInputStream.idl` (name subject to bikeshedding), so we don't have these weird constants hanging out in `Init` calls? Introduced `nsIConverterInputStream::ERRORS_ARE_FATAL`.

Pushed by hsivonen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a53f91f3fa73 Remove UTF8InputStream. r=froydnj