Re: [PATCH v2 0/1] MR7360: mshtml: Try to guess the charset on streams with no BOM.

[Jacek Caban (＠jacek)]

Sorry for the delay. In general, setting the charset for all data here is questionable, it would also apply to binary data, which doesn't make much sense.

I checked the Gecko source, and it seems that the CSS parser does rely on the protocol handler to detect the charset. This means it likely wouldn’t work for the file protocol. Did you confirm this behavior in Firefox? It should be easy to convert this test to JS and run it in stock Firefox 47 to verify that it’s not an issue with our nsio replacement.

If it turns out that we do need this behavior, perhaps we should apply it only to selected MIME types?