Re: [whatwg/encoding] Add "replacement" as a label for the replacement encoding (#70)

FYI, Blink change landed. 

Worth noting: @annevk's WPT changes didn't trip any failures when rolled into Blink's CI since the tests currently exercise the encoding labels via the API, and replacement encodings already threw just like unknown labels.

We've got [blink-specific tests that use XHR and data: URLs to verify various encodings](https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/fast/encoding/char-decoding.html?q=char-decoding.html&sq=package:chromium&l=147) and specifically that the replacement ones yield U+FFFD. I was lame and just added "replacement" to the list. We should probably tidy up and upstream those (among other fun cases like UTF-7).


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/encoding/issues/70#issuecomment-315919002

Received on Tuesday, 18 July 2017 00:24:13 UTC