On Thu Jul 7 21:06:09 2022 +0000, Jacek Caban wrote:
Those tests are specific to XHR, but your code changes the general case. Should we essentially ignore all MIME verification in all cases? BTW, there is PI_MIMEVERIFICATION that controls that on urlmon side, although I don't see an easy way to utilize that in MSHTML without using CreateBinding() directly.
But the content type does not seem to be set without a Content-Type response header (which is the reason for the first commit), so detection is still useful as a fallback in case the response does not provide one.
As for why it's specific to XHR, I thought it used the same underlying code path, but I could be wrong. Not sure how to "special case" it though. It didn't break any tests, so it's either correct or we clearly lack tests in this area. Do you have some in mind how to add tests for this (without XHR)?