diff options
author | jshin@chromium.org <jshin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-08 12:38:26 +0000 |
---|---|---|
committer | jshin@chromium.org <jshin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-08 12:38:26 +0000 |
commit | 50f9185a1132aa5727e7e79c1fae7541bdb26da8 (patch) | |
tree | 9c1c4c5f4cdded2414bee5e20b5586421b8497bf | |
parent | 5590c6f33487b5ed7724466162db56a9d3876d47 (diff) | |
download | chromium_src-50f9185a1132aa5727e7e79c1fae7541bdb26da8.zip chromium_src-50f9185a1132aa5727e7e79c1fae7541bdb26da8.tar.gz chromium_src-50f9185a1132aa5727e7e79c1fae7541bdb26da8.tar.bz2 |
Canonical encoding name : windows-949 => EUC-KR
This is the chromium side of a blink change at
https://codereview.chromium.org/25870005
Until the corresponding blink CL above is re-landed (it's landed but reverted),
disable EUC-KR-related test temporarily.
This is a copy of https://codereview.chromium.org/25769005/ (lgtm'd and approved).
BUG=277027, 95963
TEST=browser test: BrowserEncodingTest.* and EncodingAlias*.
TBR=thakis,robertshield
Review URL: https://codereview.chromium.org/26439002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227506 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/browser_encoding_browsertest.cc | 19 | ||||
-rw-r--r-- | chrome/browser/character_encoding.cc | 3 | ||||
-rw-r--r-- | chrome/test/data/encoding_tests/alias_mapping/EUC-KR.html (renamed from chrome/test/data/encoding_tests/alias_mapping/windows-949.html) | 4 | ||||
-rw-r--r-- | chrome/test/data/encoding_tests/auto_detect/EUC-KR_with_no_encoding_specified.html (renamed from chrome/test/data/encoding_tests/auto_detect/windows-949_with_no_encoding_specified.html) | 2 | ||||
-rw-r--r-- | chrome/test/data/encoding_tests/auto_detect/expected_results/expected_EUC-KR_saved_from_no_encoding_specified.html (renamed from chrome/test/data/encoding_tests/auto_detect/expected_results/expected_windows-949_saved_from_no_encoding_specified.html) | 6 | ||||
-rw-r--r-- | chrome_frame/chrome_active_document.h | 2 |
6 files changed, 22 insertions, 14 deletions
diff --git a/chrome/browser/browser_encoding_browsertest.cc b/chrome/browser/browser_encoding_browsertest.cc index e4911db..eadbfec 100644 --- a/chrome/browser/browser_encoding_browsertest.cc +++ b/chrome/browser/browser_encoding_browsertest.cc @@ -53,8 +53,12 @@ const EncodingTestData kEncodingTestDatas[] = { { "UTF-8.html", "UTF-8" }, { "UTF-16LE.html", "UTF-16LE" }, { "windows-874.html", "windows-874" }, - // http://crbug.com/95963 - // { "windows-949.html", "windows-949" }, +#if 0 + // TODO(jungshik): Temporarily disable this. Reenable it once + // Blink rolls to include https://codereview.chromium.org/25870005/ + // (it's reverted so that it has to be landed again). + { "EUC-KR.html", "EUC-KR" }, +#endif { "windows-1250.html", "windows-1250" }, { "windows-1251.html", "windows-1251" }, { "windows-1252.html", "windows-1252" }, @@ -265,9 +269,14 @@ IN_PROC_BROWSER_TEST_F(BrowserEncodingTest, MAYBE_TestEncodingAutoDetect) { { "UTF-8_with_no_encoding_specified.html", "expected_UTF-8_saved_from_no_encoding_specified.html", "UTF-8" }, - { "windows-949_with_no_encoding_specified.html", - "expected_windows-949_saved_from_no_encoding_specified.html", - "windows-949-2000" }, +#if 0 + // TODO(jungshik): Temporarily disable this. Reenable it once + // Blink rolls to include https://codereview.chromium.org/25870005/ + // (it's reverted so that it has to be landed again). + { "EUC-KR_with_no_encoding_specified.html", + "expected_EUC-KR_saved_from_no_encoding_specified.html", + "EUC-KR" }, +#endif { "windows-1251_with_no_encoding_specified.html", "expected_windows-1251_saved_from_no_encoding_specified.html", "windows-1251" }, diff --git a/chrome/browser/character_encoding.cc b/chrome/browser/character_encoding.cc index 4909a7b..7c8fef0 100644 --- a/chrome/browser/character_encoding.cc +++ b/chrome/browser/character_encoding.cc @@ -42,7 +42,7 @@ const CanonicalEncodingData kCanonicalEncodingNames[] = { { IDC_ENCODING_GB18030, "gb18030", IDS_ENCODING_SIMP_CHINESE }, { IDC_ENCODING_BIG5, "Big5", IDS_ENCODING_TRAD_CHINESE }, { IDC_ENCODING_BIG5HKSCS, "Big5-HKSCS", IDS_ENCODING_TRAD_CHINESE }, - { IDC_ENCODING_KOREAN, "windows-949", IDS_ENCODING_KOREAN }, + { IDC_ENCODING_KOREAN, "EUC-KR", IDS_ENCODING_KOREAN }, { IDC_ENCODING_SHIFTJIS, "Shift_JIS", IDS_ENCODING_JAPANESE }, { IDC_ENCODING_EUCJP, "EUC-JP", IDS_ENCODING_JAPANESE }, { IDC_ENCODING_ISO2022JP, "ISO-2022-JP", IDS_ENCODING_JAPANESE }, @@ -88,7 +88,6 @@ const CanonicalEncodingDisplayNamePair kCanonicalDisplayNameOverrides[] = { // Only lists the canonical names where we want a different form for display. { "macintosh", "Macintosh" }, { "windows-874", "Windows-874" }, - { "windows-949", "Windows-949" }, { "windows-1250", "Windows-1250" }, { "windows-1251", "Windows-1251" }, { "windows-1252", "Windows-1252" }, diff --git a/chrome/test/data/encoding_tests/alias_mapping/windows-949.html b/chrome/test/data/encoding_tests/alias_mapping/EUC-KR.html index ff2f47b..81c9655 100644 --- a/chrome/test/data/encoding_tests/alias_mapping/windows-949.html +++ b/chrome/test/data/encoding_tests/alias_mapping/EUC-KR.html @@ -1,7 +1,7 @@ <html> <head> - <title> windows-949 </title> - <meta http-equiv="Content-Type" content="text/html; charset=windows-949" /> + <title> EUC-KR </title> + <meta http-equiv="Content-Type" content="text/html; charset=EUC-KR" /> </head> <body> diff --git a/chrome/test/data/encoding_tests/auto_detect/windows-949_with_no_encoding_specified.html b/chrome/test/data/encoding_tests/auto_detect/EUC-KR_with_no_encoding_specified.html index 1803f15..1ae000e 100644 --- a/chrome/test/data/encoding_tests/auto_detect/windows-949_with_no_encoding_specified.html +++ b/chrome/test/data/encoding_tests/auto_detect/EUC-KR_with_no_encoding_specified.html @@ -1,6 +1,6 @@ <html> <head> - <title> windows-949 </title> + <title> EUC-KR </title> </head> <body> diff --git a/chrome/test/data/encoding_tests/auto_detect/expected_results/expected_windows-949_saved_from_no_encoding_specified.html b/chrome/test/data/encoding_tests/auto_detect/expected_results/expected_EUC-KR_saved_from_no_encoding_specified.html index f1351fe..161fac3 100644 --- a/chrome/test/data/encoding_tests/auto_detect/expected_results/expected_windows-949_saved_from_no_encoding_specified.html +++ b/chrome/test/data/encoding_tests/auto_detect/expected_results/expected_EUC-KR_saved_from_no_encoding_specified.html @@ -1,7 +1,7 @@ -<!-- saved from url=(0087)http://mock.http/encoding_tests/auto_detect/windows-949_with_no_encoding_specified.html --> -<html><head><meta http-equiv="Content-Type" content="text/html; charset=windows-949"> - <title> windows-949 </title> +<!-- saved from url=(0082)http://mock.http/encoding_tests/auto_detect/EUC-KR_with_no_encoding_specified.html --> +<html><head><meta http-equiv="Content-Type" content="text/html; charset=EUC-KR"> + <title> EUC-KR </title> </head> <body> diff --git a/chrome_frame/chrome_active_document.h b/chrome_frame/chrome_active_document.h index 3afcd0f..9f23b90 100644 --- a/chrome_frame/chrome_active_document.h +++ b/chrome_frame/chrome_active_document.h @@ -91,7 +91,7 @@ class ChromeActiveDocument; V(INTERNAL_IE_ENCODINGMENU_JAPAN_AUTOSELECT, 3634, "ISO-2022-JP") \ V(INTERNAL_IE_ENCODINGMENU_JAPAN_EUC, 3635, "EUC-JP") \ V(INTERNAL_IE_ENCODINGMENU_JAPAN_SHIFT_JIS, 3636, "Shift_JIS") \ - V(INTERNAL_IE_ENCODINGMENU_KOREA, 3637, "windows-949") \ + V(INTERNAL_IE_ENCODINGMENU_KOREA, 3637, "EUC-KR") \ V(INTERNAL_IE_ENCODINGMENU_THAI, 3638, "windows-874") \ V(INTERNAL_IE_ENCODINGMENU_TURKISH_ISO, 3639, "windows-1254") \ V(INTERNAL_IE_ENCODINGMENU_TURKISH_WINDOWS, 3640, "windows-1254") \ |