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

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/navigate.c:
>              break;
>          case BOM_UTF16:
>              This->nschannel->charset = strdup("utf-16");
> -
>        case BOM_NONE:
> -
>            /* FIXME: Get charset from HTTP headers */;
> -
>            break;
> -
>        case BOM_NONE: {
> -
>            /* FIXME: Get charset from HTTP headers first */

The comment is outdated, we already look at HTTP headers, but with this change, we'd leak and override the charset found there. (It's already true for cases where BOM is present, which is likely wrong, but at least the value is more reliable there.)