Title: [224747] trunk
Revision
224747
Author
[email protected]
Date
2017-11-12 23:54:55 -0800 (Sun, 12 Nov 2017)

Log Message

Remove support for UTF-32 variant text encodings
https://bugs.webkit.org/show_bug.cgi?id=179435

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

* web-platform-tests/encoding/unsupported-encodings-expected.txt: Update results now
that UTF-32 support is gone. Still not quite right since we don't sniff UTF-32LE as
UTF-16LE yet.
* web-platform-tests/encoding/utf-32-expected.txt: ditto

Source/WebCore:

Non-support for these encodings is covered by WPT tests.

* platform/text/TextCodecICU.cpp: Remove UTF-32 variants.
Technically, we should also ensure that UTF-32LE with a BOM
gets sniffed as UTF-16LE instead of windows-1252. Chrome
does this but Firefox doesn't yet. WPT tests cover it, and
we can fix it later.

LayoutTests:

* fast/encoding/legacy-utf-32-aliases.html: Added. Check that
we don't support our legacy aliases for utf-32, since WPT doesn't
either.
* fast/encoding/legacy-utf-32-aliases-expected.txt: Ditto.

Remove the following tests; we no longer want to support UTF-32.

imported/w3c/web-platform-tests/encoding already has tests for lack of support.

* fast/encoding/utf-32-big-endian-bom-expected.txt: Removed.
* fast/encoding/utf-32-big-endian-bom.html: Removed.
* fast/encoding/utf-32-big-endian-nobom-expected.txt: Removed.
* fast/encoding/utf-32-big-endian-nobom.xml: Removed.
* fast/encoding/utf-32-little-endian-bom-expected.txt: Removed.
* fast/encoding/utf-32-little-endian-bom.html: Removed.
* fast/encoding/utf-32-little-endian-nobom-expected.txt: Removed.
* fast/encoding/utf-32-little-endian-nobom.xml: Removed.
* http/tests/misc/submit-get-in-utf32be-expected.txt: Removed.
* http/tests/misc/submit-get-in-utf32be.html: Removed.
* http/tests/misc/submit-get-in-utf32le-expected.txt: Removed.
* http/tests/misc/submit-get-in-utf32le.html: Removed.
* http/tests/misc/submit-post-in-utf32be-expected.txt: Removed.
* http/tests/misc/submit-post-in-utf32be.html: Removed.
* http/tests/misc/submit-post-in-utf32le-expected.txt: Removed.
* http/tests/misc/submit-post-in-utf32le.html: Removed.
* http/tests/misc/url-in-utf32be-expected.txt: Removed.
* http/tests/misc/url-in-utf32be.html: Removed.
* http/tests/misc/url-in-utf32le-expected.txt: Removed.
* http/tests/misc/url-in-utf32le.html: Removed.

Modified Paths

Added Paths

Removed Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (224746 => 224747)


--- trunk/LayoutTests/ChangeLog	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/ChangeLog	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +1,40 @@
+2017-11-12  Maciej Stachowiak  <[email protected]>
+
+        Remove support for UTF-32 variant text encodings
+        https://bugs.webkit.org/show_bug.cgi?id=179435
+
+        Reviewed by Darin Adler.
+
+        * fast/encoding/legacy-utf-32-aliases.html: Added. Check that
+        we don't support our legacy aliases for utf-32, since WPT doesn't
+        either.
+        * fast/encoding/legacy-utf-32-aliases-expected.txt: Ditto.
+
+        Remove the following tests; we no longer want to support UTF-32. 
+        
+        imported/w3c/web-platform-tests/encoding already has tests for lack of support.
+        
+        * fast/encoding/utf-32-big-endian-bom-expected.txt: Removed.
+        * fast/encoding/utf-32-big-endian-bom.html: Removed.
+        * fast/encoding/utf-32-big-endian-nobom-expected.txt: Removed.
+        * fast/encoding/utf-32-big-endian-nobom.xml: Removed.
+        * fast/encoding/utf-32-little-endian-bom-expected.txt: Removed.
+        * fast/encoding/utf-32-little-endian-bom.html: Removed.
+        * fast/encoding/utf-32-little-endian-nobom-expected.txt: Removed.
+        * fast/encoding/utf-32-little-endian-nobom.xml: Removed.
+        * http/tests/misc/submit-get-in-utf32be-expected.txt: Removed.
+        * http/tests/misc/submit-get-in-utf32be.html: Removed.
+        * http/tests/misc/submit-get-in-utf32le-expected.txt: Removed.
+        * http/tests/misc/submit-get-in-utf32le.html: Removed.
+        * http/tests/misc/submit-post-in-utf32be-expected.txt: Removed.
+        * http/tests/misc/submit-post-in-utf32be.html: Removed.
+        * http/tests/misc/submit-post-in-utf32le-expected.txt: Removed.
+        * http/tests/misc/submit-post-in-utf32le.html: Removed.
+        * http/tests/misc/url-in-utf32be-expected.txt: Removed.
+        * http/tests/misc/url-in-utf32be.html: Removed.
+        * http/tests/misc/url-in-utf32le-expected.txt: Removed.
+        * http/tests/misc/url-in-utf32le.html: Removed.
+
 2017-11-12  Carlos Garcia Campos  <[email protected]>
 
         REGRESSION(r224566): [GTK] 26 new timeouts on inspector related tests.

Added: trunk/LayoutTests/fast/encoding/legacy-utf-32-aliases-expected.txt (0 => 224747)


--- trunk/LayoutTests/fast/encoding/legacy-utf-32-aliases-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/fast/encoding/legacy-utf-32-aliases-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -0,0 +1,22 @@
+This test verifies that certain aliases for UTF-32 are not supported.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+The following encodings should not be supported
+PASS new TextDecoder("ISO-10646-UCS-4").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-1236").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-1237").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("csUCS4").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ucs-4").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("UTF32_LittleEndian").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-1234").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-1235").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("UTF32_BigEndian").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-1232").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-1233").encoding threw exception RangeError: Bad value.
+PASS new TextDecoder("ibm-9424").encoding threw exception RangeError: Bad value.
+PASS successfullyParsed is true
+
+TEST COMPLETE
+

Added: trunk/LayoutTests/fast/encoding/legacy-utf-32-aliases.html (0 => 224747)


--- trunk/LayoutTests/fast/encoding/legacy-utf-32-aliases.html	                        (rev 0)
+++ trunk/LayoutTests/fast/encoding/legacy-utf-32-aliases.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -0,0 +1,24 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta charset="utf-8">
+<script src=""
+</head>
+<body>
+<script>
+description("This test verifies that certain aliases for UTF-32 are not supported.");
+
+debug("The following encodings should not be supported");
+
+let utf32LegacyAliases = [
+"ISO-10646-UCS-4", "ibm-1236", "ibm-1237", "csUCS4", "ucs-4", "UTF32_LittleEndian", "ibm-1234", "ibm-1235", "UTF32_BigEndian", "ibm-1232", "ibm-1233", "ibm-9424"
+]; 
+
+for (let encoding of utf32LegacyAliases) {
+    let canonical_name_expr = 'new TextDecoder("' + encoding + '").encoding';
+    shouldThrow(canonical_name_expr);
+}
+
+</script>
+</body>
+</html>

Deleted: trunk/LayoutTests/fast/encoding/utf-32-big-endian-bom-expected.txt (224746 => 224747)


--- trunk/LayoutTests/fast/encoding/utf-32-big-endian-bom-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/fast/encoding/utf-32-big-endian-bom-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,5 +0,0 @@
-When dumped into a text file by the test controller, this needs to be viewed as UTF-8:
-
-This is an em dash, —, this is is a delta symbol, ∆, this is an uppercase pi, ∏, and this is a lowercase pi, π.
-
-Success : UTF-32BE

Deleted: trunk/LayoutTests/fast/encoding/utf-32-big-endian-bom.html


(Binary files differ)

Deleted: trunk/LayoutTests/fast/encoding/utf-32-big-endian-nobom-expected.txt (224746 => 224747)


--- trunk/LayoutTests/fast/encoding/utf-32-big-endian-nobom-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/fast/encoding/utf-32-big-endian-nobom-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,5 +0,0 @@
-When dumped into a text file by the test controller, this needs to be viewed as UTF-8:
-
-This is an em dash, —, this is is a delta symbol, ∆, this is an uppercase pi, ∏, and this is a lowercase pi, π.
-
-Success : UTF-32BE

Deleted: trunk/LayoutTests/fast/encoding/utf-32-big-endian-nobom.xml


(Binary files differ)

Deleted: trunk/LayoutTests/fast/encoding/utf-32-little-endian-bom-expected.txt (224746 => 224747)


--- trunk/LayoutTests/fast/encoding/utf-32-little-endian-bom-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/fast/encoding/utf-32-little-endian-bom-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,5 +0,0 @@
-When dumped into a text file by the test controller, this needs to be viewed as UTF-8:
-
-This is an em dash, —, this is is a delta symbol, ∆, this is an uppercase pi, ∏, and this is a lowercase pi, π.
-
-Success : UTF-32LE

Deleted: trunk/LayoutTests/fast/encoding/utf-32-little-endian-bom.html


(Binary files differ)

Deleted: trunk/LayoutTests/fast/encoding/utf-32-little-endian-nobom-expected.txt (224746 => 224747)


--- trunk/LayoutTests/fast/encoding/utf-32-little-endian-nobom-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/fast/encoding/utf-32-little-endian-nobom-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,5 +0,0 @@
-When dumped into a text file by the test controller, this needs to be viewed as UTF-8:
-
-This is an em dash, —, this is is a delta symbol, ∆, this is an uppercase pi, ∏, and this is a lowercase pi, π.
-
-Success : UTF-32LE

Deleted: trunk/LayoutTests/fast/encoding/utf-32-little-endian-nobom.xml


(Binary files differ)

Deleted: trunk/LayoutTests/http/tests/misc/submit-get-in-utf32be-expected.txt (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-get-in-utf32be-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-get-in-utf32be-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +0,0 @@
-This test is for bug 21635. The query parameter in non-UTF-8 Unicode pages (UTF-7,16,32) should be converted to UTF-8 before a request is made to a server.
-
-SUCCESS: query param is converted to UTF-8

Deleted: trunk/LayoutTests/http/tests/misc/submit-get-in-utf32be.html (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-get-in-utf32be.html	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-get-in-utf32be.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,21 +0,0 @@
-����\xFE\xFF������<������h������t������m������l������>������
-������<������h������e������a������d������>������
-������<������s������c������r������i������p������t������>������
-������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������ ������{������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������w������a������i������t������U������n������t������i������l������D������o������n������e������(������)������;������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������d������u������m������p������A������s������T������e������x������t������(������)������;������
-������}������
-������
-������f������u������n������c������t������i������o������n������ ������c������l������i������c������k������f������(������)������
-������{������
-������ ������ ������ ������ ������d������o������c������u������m������e������n������t������.������f������.������s������u������b������m������i������t������(������)������;������
-������ ������ ������ ������ ������r������e������t������u������r������n������ ������t������r������u������e������;������
-������}������
-������<������/������s������c������r������i������p������t������>������
-������<������/������h������e������a������d������>������
-������<������b������o������d������y������ ������o������n������l������o������a������d������=������"������c������l������i������c������k������f������(������)������"������>������
-������<������f������o������r������m������ ������n������a������m������e������=������"������f������"������ ������a������c������t������i������o������n������=������"������r������e������s������o������u������r������c������e������s������/������c������h������e������c������k������-������q������u������e������r������y������-������p������a������r������a������m������.������p������h������p������"������ ������m������e������t������h������o������d������=������"������g������e������t������"������>������
-������<������i������n������p������u������t������ ������n������a������m������e������=������"������q������"������ ������t������y������p������e������=������"������t������e������x������t������"������ ������v������a������l������u������e������=������"������&������#������x������2������1������2������2������;������&������#������x������5������3������4������1������;������"������>������
-������<������/������f������o������r������m������>������
-������<������/������b������o������d������y������>������
-������<������/������h������t������m������l������>������

Deleted: trunk/LayoutTests/http/tests/misc/submit-get-in-utf32le-expected.txt (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-get-in-utf32le-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-get-in-utf32le-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +0,0 @@
-This test is for bug 21635. The query parameter in non-UTF-8 Unicode pages (UTF-7,16,32) should be converted to UTF-8 before a request is made to a server.
-
-SUCCESS: query param is converted to UTF-8

Deleted: trunk/LayoutTests/http/tests/misc/submit-get-in-utf32le.html (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-get-in-utf32le.html	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-get-in-utf32le.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,22 +0,0 @@
-\xFF\xFE����<������h������t������m������l������>������
-������<������h������e������a������d������>������
-������<������s������c������r������i������p������t������>������
-������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������ ������{������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������w������a������i������t������U������n������t������i������l������D������o������n������e������(������)������;������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������d������u������m������p������A������s������T������e������x������t������(������)������;������
-������}������
-������
-������f������u������n������c������t������i������o������n������ ������c������l������i������c������k������f������(������)������
-������{������
-������ ������ ������ ������ ������d������o������c������u������m������e������n������t������.������f������.������s������u������b������m������i������t������(������)������;������
-������ ������ ������ ������ ������r������e������t������u������r������n������ ������t������r������u������e������;������
-������}������
-������<������/������s������c������r������i������p������t������>������
-������<������/������h������e������a������d������>������
-������<������b������o������d������y������ ������o������n������l������o������a������d������=������"������c������l������i������c������k������f������(������)������"������>������
-������<������f������o������r������m������ ������n������a������m������e������=������"������f������"������ ������a������c������t������i������o������n������=������"������r������e������s������o������u������r������c������e������s������/������c������h������e������c������k������-������q������u������e������r������y������-������p������a������r������a������m������.������p������h������p������"������ ������m������e������t������h������o������d������=������"������g������e������t������"������>������
-������<������i������n������p������u������t������ ������n������a������m������e������=������"������q������"������ ������t������y������p������e������=������"������t������e������x������t������"������ ������v������a������l������u������e������=������"������&������#������x������2������1������2������2������;������&������#������x������5������3������4������1������;������"������>������
-������<������/������f������o������r������m������>������
-������<������/������b������o������d������y������>������
-������<������/������h������t������m������l������>������
-������
\ No newline at end of file

Deleted: trunk/LayoutTests/http/tests/misc/submit-post-in-utf32be-expected.txt (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-post-in-utf32be-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-post-in-utf32be-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +0,0 @@
-This test is for bug 21635. The query parameter in non-UTF-8 Unicode pages (UTF-7,16,32) should be converted to UTF-8 before a request is made to a server.
-
-SUCCESS: query param is converted to UTF-8

Deleted: trunk/LayoutTests/http/tests/misc/submit-post-in-utf32be.html (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-post-in-utf32be.html	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-post-in-utf32be.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,19 +0,0 @@
-����\xFE\xFF������<������h������t������m������l������>������
-������<������h������e������a������d������>������
-������<������s������c������r������i������p������t������>������
-������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������ ������{������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������w������a������i������t������U������n������t������i������l������D������o������n������e������(������)������;������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������d������u������m������p������A������s������T������e������x������t������(������)������;������
-������}������
-������f������u������n������c������t������i������o������n������ ������c������l������i������c������k������f������(������)������
-������{������
-������ ������ ������ ������ ������d������o������c������u������m������e������n������t������.������f������.������s������u������b������m������i������t������(������)������;������
-������}������
-������<������/������s������c������r������i������p������t������>������
-������<������/������h������e������a������d������>������
-������<������b������o������d������y������ ������o������n������l������o������a������d������=������"������c������l������i������c������k������f������(������)������"������>������
-������<������f������o������r������m������ ������n������a������m������e������=������"������f������"������ ������a������c������t������i������o������n������=������"������r������e������s������o������u������r������c������e������s������/������c������h������e������c������k������-������q������u������e������r������y������-������p������a������r������a������m������.������p������h������p������"������ ������m������e������t������h������o������d������=������"������p������o������s������t������"������>������
-������<������i������n������p������u������t������ ������n������a������m������e������=������"������q������"������ ������t������y������p������e������=������"������t������e������x������t������"������ ������v������a������l������u������e������=������"������&������#������x������2������1������2������2������;������&������#������x������5������3������4������1������;������"������>������
-������<������/������f������o������r������m������>������
-������<������/������b������o������d������y������>������
-������<������/������h������t������m������l������>������

Deleted: trunk/LayoutTests/http/tests/misc/submit-post-in-utf32le-expected.txt (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-post-in-utf32le-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-post-in-utf32le-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +0,0 @@
-This test is for bug 21635. The query parameter in non-UTF-8 Unicode pages (UTF-7,16,32) should be converted to UTF-8 before a request is made to a server.
-
-SUCCESS: query param is converted to UTF-8

Deleted: trunk/LayoutTests/http/tests/misc/submit-post-in-utf32le.html (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/submit-post-in-utf32le.html	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/submit-post-in-utf32le.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,20 +0,0 @@
-\xFF\xFE����<������h������t������m������l������>������
-������<������h������e������a������d������>������
-������<������s������c������r������i������p������t������>������
-������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������ ������{������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������w������a������i������t������U������n������t������i������l������D������o������n������e������(������)������;������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������d������u������m������p������A������s������T������e������x������t������(������)������;������
-������}������
-������f������u������n������c������t������i������o������n������ ������c������l������i������c������k������f������(������)������
-������{������
-������ ������ ������ ������ ������d������o������c������u������m������e������n������t������.������f������.������s������u������b������m������i������t������(������)������;������
-������}������
-������<������/������s������c������r������i������p������t������>������
-������<������/������h������e������a������d������>������
-������<������b������o������d������y������ ������o������n������l������o������a������d������=������"������c������l������i������c������k������f������(������)������"������>������
-������<������f������o������r������m������ ������n������a������m������e������=������"������f������"������ ������a������c������t������i������o������n������=������"������r������e������s������o������u������r������c������e������s������/������c������h������e������c������k������-������q������u������e������r������y������-������p������a������r������a������m������.������p������h������p������"������ ������m������e������t������h������o������d������=������"������p������o������s������t������"������>������
-������<������i������n������p������u������t������ ������n������a������m������e������=������"������q������"������ ������t������y������p������e������=������"������t������e������x������t������"������ ������v������a������l������u������e������=������"������&������#������x������2������1������2������2������;������&������#������x������5������3������4������1������;������"������>������
-������<������/������f������o������r������m������>������
-������<������/������b������o������d������y������>������
-������<������/������h������t������m������l������>������
-������
\ No newline at end of file

Deleted: trunk/LayoutTests/http/tests/misc/url-in-utf32be-expected.txt (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/url-in-utf32be-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/url-in-utf32be-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,2 +0,0 @@
-This test is for bug 21635. The query parameter in non-UTF-8 Unicode pages (UTF-7,16,32) should be converted to UTF-8 before a request is made to a server.
-SUCCESS: query param is converted to UTF-8

Deleted: trunk/LayoutTests/http/tests/misc/url-in-utf32be.html (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/url-in-utf32be.html	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/url-in-utf32be.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,30 +0,0 @@
-����\xFE\xFF������<������h������t������m������l������>������
-������<������h������e������a������d������>������
-������<������s������c������r������i������p������t������>������
-������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������ ������{������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������d������u������m������p������A������s������T������e������x������t������(������)������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������w������a������i������t������U������n������t������i������l������D������o������n������e������(������)������;������
-������}������
-������
-������f������u������n������c������t������i������o������n������ ������r������u������n������T������e������s������t������(������)������
-������{������
-������ ������ ������ ������ ������v������a������r������ ������f������ ������=������ ������d������o������c������u������m������e������n������t������.������g������e������t������E������l������e������m������e������n������t������B������y������I������d������(������'������t������h������e������F������r������a������m������e������'������)������;������
-������ ������ ������ ������ ������v������a������r������ ������r������ ������=������ ������d������o������c������u������m������e������n������t������.������g������e������t������E������l������e������m������e������n������t������B������y������I������d������(������'������r������e������s������u������l������t������'������)������;������
-������ ������ ������ ������ ������v������a������r������ ������o������ ������=������ ������f������.������c������o������n������t������e������n������t������D������o������c������u������m������e������n������t������.������g������e������t������E������l������e������m������e������n������t������B������y������I������d������(������'������o������u������t������p������u������t������'������)������.������f������i������r������s������t������C������h������i������l������d������;������
-������ ������ ������ ������ ������i������f������ ������(������o������.������n������o������d������e������V������a������l������u������e������ ������=������=������ ������'������\������u������2������1������2������2������\������u������5������3������4������1������'������)������ ������
-������ ������ ������ ������ ������ ������ ������ ������ ������r������.������i������n������n������e������r������H������T������M������L������ ������=������ ������"������S������U������C������C������E������S������S������:������ ������q������u������e������r������y������ ������p������a������r������a������m������ ������i������s������ ������c������o������n������v������e������r������t������e������d������ ������t������o������ ������U������T������F������-������8������"������;������
-������ ������ ������ ������ ������e������l������s������e������
-������ ������ ������ ������ ������ ������ ������ ������ ������r������.������i������n������n������e������r������H������T������M������L������ ������=������ ������"������F������A������I������L������U������R������E������:������ ������q������u������e������r������y������ ������p������a������r������a������m������ ������i������s������ ������n������o������t������ ������c������o������n������v������e������r������t������e������d������ ������t������o������ ������U������T������F������-������8������.������ ������v������a������l������u������e������=������"������ ������+������
-������ ������ ������ ������ ������ ������ ������ ������ ������o������.������n������o������d������e������V������a������l������u������e������;������
-������ ������ ������ ������ ������ ������ ������ ������ ������
-������ ������ ������ ������ ������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������
-������ ������ ������ ������ ������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������n������o������t������i������f������y������D������o������n������e������(������)������;������
-������}������
-������<������/������s������c������r������i������p������t������>������
-������<������/������h������e������a������d������>������
-������<������b������o������d������y������ ������o������n������l������o������a������d������=������"������r������u������n������T������e������s������t������(������)������"������>������
-������<������i������f������r������a������m������e������ ������s������t������y������l������e������=������"������d������i������s������p������l������a������y������:������ ������n������o������n������e������;������"������ ������i������d������=������"������t������h������e������F������r������a������m������e������"������ ������s������r������c������=������"������r������e������s������o������u������r������c������e������s������/������e������c������h������o������-������q������u������e������r������y������-������p������a������r������a������m������.������p������h������p������?������q������=������&������#������x������2������1������2������2������;������&������#������x������5������3������4������1������;������"������>������<������/������i������f������r������a������m������e������>������
-������<������d������i������v������>������T������h������i������s������ ������t������e������s������t������ ������i������s������ ������f������o������r������ ������<������a������ ������h������r������e������f������=������"������h������t������t������p������s������:������/������/������b������u������g������s������.������w������e������b������k������i������t������.������o������r������g������/������s������h������o������w������_������b������u������g������.������c������g������i������?������i������d������=������2������1������6������3������5������"������>������b������u������g������ ������2������1������6������3������5������<������/������a������>������.������ ������T������h������e������ ������q������u������e������r������y������ ������p������a������r������a������m������e������t������e������r������ ������i������n������ ������n������o������n������-������U������T������F������-������8������ ������U������n������i������c������o������d������e������ ������p������a������g������e���
 ���s������ ������(������U������T������F������-������7������,������1������6������,������3������2������)������ ������
-������s������h������o������u������l������d������ ������b������e������ ������c������o������n������v������e������r������t������e������d������ ������t������o������ ������U������T������F������-������8������ ������b������e������f������o������r������e������ ������a������ ������r������e������q������u������e������s������t������ ������i������s������ ������m������a������d������e������ ������t������o������ ������a������ ������s������e������r������v������e������r������.������
-������<������d������i������v������ ������i������d������=������"������r������e������s������u������l������t������"������>������<������/������d������i������v������>������
-������<������/������h������t������m������l������>������

Deleted: trunk/LayoutTests/http/tests/misc/url-in-utf32le-expected.txt (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/url-in-utf32le-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/url-in-utf32le-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,2 +0,0 @@
-This test is for bug 21635. The query parameter in non-UTF-8 Unicode pages (UTF-7,16,32) should be converted to UTF-8 before a request is made to a server.
-SUCCESS: query param is converted to UTF-8

Deleted: trunk/LayoutTests/http/tests/misc/url-in-utf32le.html (224746 => 224747)


--- trunk/LayoutTests/http/tests/misc/url-in-utf32le.html	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/http/tests/misc/url-in-utf32le.html	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,31 +0,0 @@
-\xFF\xFE����<������h������t������m������l������>������
-������<������h������e������a������d������>������
-������<������s������c������r������i������p������t������>������
-������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������ ������{������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������d������u������m������p������A������s������T������e������x������t������(������)������
-������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������w������a������i������t������U������n������t������i������l������D������o������n������e������(������)������;������
-������}������
-������
-������f������u������n������c������t������i������o������n������ ������r������u������n������T������e������s������t������(������)������
-������{������
-������ ������ ������ ������ ������v������a������r������ ������f������ ������=������ ������d������o������c������u������m������e������n������t������.������g������e������t������E������l������e������m������e������n������t������B������y������I������d������(������'������t������h������e������F������r������a������m������e������'������)������;������
-������ ������ ������ ������ ������v������a������r������ ������r������ ������=������ ������d������o������c������u������m������e������n������t������.������g������e������t������E������l������e������m������e������n������t������B������y������I������d������(������'������r������e������s������u������l������t������'������)������;������
-������ ������ ������ ������ ������v������a������r������ ������o������ ������=������ ������f������.������c������o������n������t������e������n������t������D������o������c������u������m������e������n������t������.������g������e������t������E������l������e������m������e������n������t������B������y������I������d������(������'������o������u������t������p������u������t������'������)������.������f������i������r������s������t������C������h������i������l������d������;������
-������ ������ ������ ������ ������i������f������ ������(������o������.������n������o������d������e������V������a������l������u������e������ ������=������=������ ������'������\������u������2������1������2������2������\������u������5������3������4������1������'������)������ ������
-������ ������ ������ ������ ������ ������ ������ ������ ������r������.������i������n������n������e������r������H������T������M������L������ ������=������ ������"������S������U������C������C������E������S������S������:������ ������q������u������e������r������y������ ������p������a������r������a������m������ ������i������s������ ������c������o������n������v������e������r������t������e������d������ ������t������o������ ������U������T������F������-������8������"������;������
-������ ������ ������ ������ ������e������l������s������e������
-������ ������ ������ ������ ������ ������ ������ ������ ������r������.������i������n������n������e������r������H������T������M������L������ ������=������ ������"������F������A������I������L������U������R������E������:������ ������q������u������e������r������y������ ������p������a������r������a������m������ ������i������s������ ������n������o������t������ ������c������o������n������v������e������r������t������e������d������ ������t������o������ ������U������T������F������-������8������.������ ������v������a������l������u������e������=������"������ ������+������
-������ ������ ������ ������ ������ ������ ������ ������ ������o������.������n������o������d������e������V������a������l������u������e������;������
-������ ������ ������ ������ ������ ������ ������ ������ ������
-������ ������ ������ ������ ������i������f������ ������(������w������i������n������d������o������w������.������t������e������s������t������R������u������n������n������e������r������)������
-������ ������ ������ ������ ������ ������ ������ ������ ������t������e������s������t������R������u������n������n������e������r������.������n������o������t������i������f������y������D������o������n������e������(������)������;������
-������}������
-������<������/������s������c������r������i������p������t������>������
-������<������/������h������e������a������d������>������
-������<������b������o������d������y������ ������o������n������l������o������a������d������=������"������r������u������n������T������e������s������t������(������)������"������>������
-������<������i������f������r������a������m������e������ ������s������t������y������l������e������=������"������d������i������s������p������l������a������y������:������ ������n������o������n������e������;������"������ ������i������d������=������"������t������h������e������F������r������a������m������e������"������ ������s������r������c������=������"������r������e������s������o������u������r������c������e������s������/������e������c������h������o������-������q������u������e������r������y������-������p������a������r������a������m������.������p������h������p������?������q������=������&������#������x������2������1������2������2������;������&������#������x������5������3������4������1������;������"������>������<������/������i������f������r������a������m������e������>������
-������<������d������i������v������>������T������h������i������s������ ������t������e������s������t������ ������i������s������ ������f������o������r������ ������<������a������ ������h������r������e������f������=������"������h������t������t������p������s������:������/������/������b������u������g������s������.������w������e������b������k������i������t������.������o������r������g������/������s������h������o������w������_������b������u������g������.������c������g������i������?������i������d������=������2������1������6������3������5������"������>������b������u������g������ ������2������1������6������3������5������<������/������a������>������.������ ������T������h������e������ ������q������u������e������r������y������ ������p������a������r������a������m������e������t������e������r������ ������i������n������ ������n������o������n������-������U������T������F������-������8������ ������U������n������i������c������o������d������e������ ������p������a������g������e���
 ���s������ ������(������U������T������F������-������7������,������1������6������,������3������2������)������ ������
-������s������h������o������u������l������d������ ������b������e������ ������c������o������n������v������e������r������t������e������d������ ������t������o������ ������U������T������F������-������8������ ������b������e������f������o������r������e������ ������a������ ������r������e������q������u������e������s������t������ ������i������s������ ������m������a������d������e������ ������t������o������ ������a������ ������s������e������r������v������e������r������.������
-������<������d������i������v������ ������i������d������=������"������r������e������s������u������l������t������"������>������<������/������d������i������v������>������
-������<������/������h������t������m������l������>������
-������
\ No newline at end of file

Modified: trunk/LayoutTests/imported/w3c/ChangeLog (224746 => 224747)


--- trunk/LayoutTests/imported/w3c/ChangeLog	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +1,15 @@
+2017-11-12  Maciej Stachowiak  <[email protected]>
+
+        Remove support for UTF-32 variant text encodings
+        https://bugs.webkit.org/show_bug.cgi?id=179435
+
+        Reviewed by Darin Adler.
+
+        * web-platform-tests/encoding/unsupported-encodings-expected.txt: Update results now 
+        that UTF-32 support is gone. Still not quite right since we don't sniff UTF-32LE as 
+        UTF-16LE yet.
+        * web-platform-tests/encoding/utf-32-expected.txt: ditto
+
 2017-11-11  Chris Dumez  <[email protected]>
 
         Re-sync Service Workers web-platform-tests from upstream

Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/encoding/unsupported-encodings-expected.txt (224746 => 224747)


--- trunk/LayoutTests/imported/w3c/web-platform-tests/encoding/unsupported-encodings-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/encoding/unsupported-encodings-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,16 +1,16 @@
 
 PASS UTF-7 should not be supported 
 PASS utf-7 should not be supported 
-FAIL UTF-32 with BOM should decode as UTF-16LE assert_equals: Decoding with UTF-32 expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0042"
-FAIL UTF-32 with no BOM should decode as windows-1252 assert_equals: Decoding with UTF-32 expected "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000" but got "U+FFFD/U+FFFD"
-FAIL utf-32 with BOM should decode as UTF-16LE assert_equals: Decoding with utf-32 expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0042"
-FAIL utf-32 with no BOM should decode as windows-1252 assert_equals: Decoding with utf-32 expected "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000" but got "U+FFFD/U+FFFD"
-FAIL UTF-32LE with BOM should decode as UTF-16LE assert_equals: Decoding with UTF-32LE expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0042"
-FAIL UTF-32LE with no BOM should decode as windows-1252 assert_equals: Decoding with UTF-32LE expected "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000" but got "U+0041/U+0042"
-FAIL utf-32le with BOM should decode as UTF-16LE assert_equals: Decoding with utf-32le expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0042"
-FAIL utf-32le with no BOM should decode as windows-1252 assert_equals: Decoding with utf-32le expected "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000" but got "U+0041/U+0042"
-FAIL UTF-32be with no BOM should decode as windows-1252 assert_equals: Decoding with UTF-32be expected "U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042" but got "U+0041/U+0042"
-FAIL UTF-32be with BOM should decode as windows-1252 assert_equals: Decoding with UTF-32be expected "U+0000/U+0000/U+00FE/U+00FF/U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042" but got "U+0041/U+0042"
-FAIL utf-32be with no BOM should decode as windows-1252 assert_equals: Decoding with utf-32be expected "U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042" but got "U+0041/U+0042"
-FAIL utf-32be with BOM should decode as windows-1252 assert_equals: Decoding with utf-32be expected "U+0000/U+0000/U+00FE/U+00FF/U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042" but got "U+0041/U+0042"
+FAIL UTF-32 with BOM should decode as UTF-16LE assert_equals: Decoding with UTF-32 expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000"
+PASS UTF-32 with no BOM should decode as windows-1252 
+FAIL utf-32 with BOM should decode as UTF-16LE assert_equals: Decoding with utf-32 expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000"
+PASS utf-32 with no BOM should decode as windows-1252 
+FAIL UTF-32LE with BOM should decode as UTF-16LE assert_equals: Decoding with UTF-32LE expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000"
+PASS UTF-32LE with no BOM should decode as windows-1252 
+FAIL utf-32le with BOM should decode as UTF-16LE assert_equals: Decoding with utf-32le expected "U+0000/U+0041/U+0000/U+0042/U+0000" but got "U+0041/U+0000/U+0000/U+0000/U+0042/U+0000/U+0000/U+0000"
+PASS utf-32le with no BOM should decode as windows-1252 
+PASS UTF-32be with no BOM should decode as windows-1252 
+FAIL UTF-32be with BOM should decode as windows-1252 assert_equals: Decoding with UTF-32be expected "U+0000/U+0000/U+00FE/U+00FF/U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042" but got "U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042"
+PASS utf-32be with no BOM should decode as windows-1252 
+FAIL utf-32be with BOM should decode as windows-1252 assert_equals: Decoding with utf-32be expected "U+0000/U+0000/U+00FE/U+00FF/U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042" but got "U+0000/U+0000/U+0000/U+0041/U+0000/U+0000/U+0000/U+0042"
 

Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/encoding/utf-32-expected.txt (224746 => 224747)


--- trunk/LayoutTests/imported/w3c/web-platform-tests/encoding/utf-32-expected.txt	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/encoding/utf-32-expected.txt	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,10 +1,10 @@
 
-FAIL Expect resources/utf-32-big-endian-bom.html to parse as windows-1252 assert_equals: expected "windows-1252" but got "UTF-32BE"
-FAIL Expect resources/utf-32-big-endian-bom.xml to parse as windows-1252 assert_equals: expected "windows-1252" but got "UTF-32BE"
+PASS Expect resources/utf-32-big-endian-bom.html to parse as windows-1252 
+PASS Expect resources/utf-32-big-endian-bom.xml to parse as windows-1252 
 PASS Expect resources/utf-32-big-endian-nobom.html to parse as windows-1252 
-FAIL Expect resources/utf-32-big-endian-nobom.xml to parse as windows-1252 assert_equals: expected "windows-1252" but got "UTF-32BE"
-FAIL Expect resources/utf-32-little-endian-bom.html to parse as UTF-16LE assert_equals: expected "UTF-16LE" but got "UTF-32LE"
-FAIL Expect resources/utf-32-little-endian-bom.xml to parse as UTF-16LE assert_equals: expected "UTF-16LE" but got "UTF-32LE"
+PASS Expect resources/utf-32-big-endian-nobom.xml to parse as windows-1252 
+FAIL Expect resources/utf-32-little-endian-bom.html to parse as UTF-16LE assert_equals: expected "UTF-16LE" but got "windows-1252"
+FAIL Expect resources/utf-32-little-endian-bom.xml to parse as UTF-16LE assert_equals: expected "UTF-16LE" but got "windows-1252"
 PASS Expect resources/utf-32-little-endian-nobom.html to parse as windows-1252 
-FAIL Expect resources/utf-32-little-endian-nobom.xml to parse as windows-1252 assert_equals: expected "windows-1252" but got "UTF-32LE"
+PASS Expect resources/utf-32-little-endian-nobom.xml to parse as windows-1252 
 

Modified: trunk/Source/WebCore/ChangeLog (224746 => 224747)


--- trunk/Source/WebCore/ChangeLog	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/Source/WebCore/ChangeLog	2017-11-13 07:54:55 UTC (rev 224747)
@@ -1,3 +1,18 @@
+2017-11-12  Maciej Stachowiak  <[email protected]>
+
+        Remove support for UTF-32 variant text encodings
+        https://bugs.webkit.org/show_bug.cgi?id=179435
+
+        Reviewed by Darin Adler.
+
+        Non-support for these encodings is covered by WPT tests.
+
+        * platform/text/TextCodecICU.cpp: Remove UTF-32 variants.
+        Technically, we should also ensure that UTF-32LE with a BOM
+        gets sniffed as UTF-16LE instead of windows-1252. Chrome
+        does this but Firefox doesn't yet. WPT tests cover it, and
+        we can fix it later.
+
 2017-11-12  Zan Dobersek  <[email protected]>
 
         [Cairo] Move rectangle and path filling, stroking, clearing operations in GraphicsContextCairo to CairoOperations

Modified: trunk/Source/WebCore/platform/text/TextCodecICU.cpp (224746 => 224747)


--- trunk/Source/WebCore/platform/text/TextCodecICU.cpp	2017-11-13 07:43:22 UTC (rev 224746)
+++ trunk/Source/WebCore/platform/text/TextCodecICU.cpp	2017-11-13 07:54:55 UTC (rev 224747)
@@ -99,9 +99,6 @@
 DECLARE_ALIASES(ISO_2022_JP, "jis7", "csiso2022jp");
 DECLARE_ALIASES(Shift_JIS, "shift-jis", "csshiftjis", "ms932", "ms_kanji", "sjis", "windows-31j", "x-sjis");
 // Encodings below are not in the standard.
-DECLARE_ALIASES(UTF_32, "ISO-10646-UCS-4", "ibm-1236", "ibm-1237", "csUCS4", "ucs-4");
-DECLARE_ALIASES(UTF_32LE, "UTF32_LittleEndian", "ibm-1234", "ibm-1235");
-DECLARE_ALIASES(UTF_32BE, "UTF32_BigEndian", "ibm-1232", "ibm-1233", "ibm-9424");
 DECLARE_ALIASES(x_mac_greek, "windows-10006", "macgr", "x-MacGreek");
 DECLARE_ALIASES(x_mac_centraleurroman, "windows-10029", "x-mac-ce", "macce", "maccentraleurope", "x-MacCentralEurope");
 DECLARE_ALIASES(x_mac_turkish, "windows-10081", "mactr", "x-MacTurkish");
@@ -153,9 +150,6 @@
     DECLARE_ENCODING_NAME("ISO-2022-JP", ISO_2022_JP),
     DECLARE_ENCODING_NAME("Shift_JIS", Shift_JIS),
     // Encodings below are not in the standard.
-    DECLARE_ENCODING_NAME("UTF-32", UTF_32),
-    DECLARE_ENCODING_NAME("UTF-32LE", UTF_32LE),
-    DECLARE_ENCODING_NAME("UTF-32BE", UTF_32BE),
     DECLARE_ENCODING_NAME("x-mac-greek", x_mac_greek),
     DECLARE_ENCODING_NAME("x-mac-centraleurroman", x_mac_centraleurroman),
     DECLARE_ENCODING_NAME("x-mac-turkish", x_mac_turkish),
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to