Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(37)

Unified Diff: LayoutTests/fast/encoding/api/resources/shared.js

Issue 145973021: Implement "replacement" text encoding. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Reworked tests, still WIP Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/encoding/api/resources/shared.js
diff --git a/LayoutTests/fast/encoding/api/resources/shared.js b/LayoutTests/fast/encoding/api/resources/shared.js
index e5874ca8f0d94d441d011dc38917f83d4f5ccf2f..530183c59b6abd260df54c0537c812f1b9be048b 100644
--- a/LayoutTests/fast/encoding/api/resources/shared.js
+++ b/LayoutTests/fast/encoding/api/resources/shared.js
@@ -1,8 +1,20 @@
+// This file is based on non-normative encodings.json resource referenced by
+// http://encoding.spec.whatwg.org/ - a reference copy is saved in this
+// directory and can be updated via:
+// curl -O http://encoding.spec.whatwg.org/encodings.json
+//
+// Changes made to this file are:
+// * whitespace
+// * encodings.json data assigned to `encodings_table` (for tests)
+// * UTF encodings listed in `utf_encodings` (for tests)
+// * 'ibm866' not yet supported - crbug.com/277023
+// * 'gb18030' distinct from 'gbk' - crbug.com/339862
+// * 'hz-gb-2312' is replacement label - w3.org/Bugs/Public/show_bug.cgi?id=25339
+
// Only these encodings are supported for encoding (vs. decoding)
-utf_encodings = ["utf-8", "utf-16le", "utf-16be"];
+var utf_encodings = ["utf-8", "utf-16le", "utf-16be"];
-// From non-normative encodings.json resource referenced by http://encoding.spec.whatwg.org/
-encodings_table = [
+var encodings_table = [
{
"encodings": [
{
@@ -346,17 +358,12 @@ encodings_table = [
],
"name": "gbk"
},
+ // 'gb18030' separate from 'gbk': crbug.com/339862
{
"labels": [
"gb18030"
],
"name": "gb18030"
- },
- {
- "labels": [
- "hz-gb-2312"
- ],
- "name": "hz-gb-2312"
}
],
"heading": "Legacy multi-byte Chinese (simplified) encodings"
@@ -424,13 +431,6 @@ encodings_table = [
"windows-949"
],
"name": "euc-kr"
- },
- {
- "labels": [
- "csiso2022kr",
- "iso-2022-kr"
- ],
- "name": "iso-2022-kr"
}
],
"heading": "Legacy multi-byte Korean encodings"
@@ -439,8 +439,12 @@ encodings_table = [
"encodings": [
{
"labels": [
+ "csiso2022kr",
+ // 'hz-gb-2312' added: w3.org/Bugs/Public/show_bug.cgi?id=25339
+ "hz-gb-2312",
"iso-2022-cn",
- "iso-2022-cn-ext"
+ "iso-2022-cn-ext",
+ "iso-2022-kr"
],
"name": "replacement"
},
« no previous file with comments | « LayoutTests/fast/encoding/api/resources/encodings.json ('k') | LayoutTests/fast/encoding/char-decoding.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698