summaryrefslogtreecommitdiffstats
path: root/chrome/browser/browser_encoding_browsertest.cc
diff options
context:
space:
mode:
authorjshin <jshin@chromium.org>2014-10-21 23:51:29 -0700
committerCommit bot <commit-bot@chromium.org>2014-10-22 06:52:48 +0000
commit0f97763f80b69d6bb3fb974c63024de93e518c74 (patch)
tree0a1a99286f8d6211611a7c1c19550abcbb31b832 /chrome/browser/browser_encoding_browsertest.cc
parent161168015de91abe74ba7be393a12a63f99a0636 (diff)
downloadchromium_src-0f97763f80b69d6bb3fb974c63024de93e518c74.zip
chromium_src-0f97763f80b69d6bb3fb974c63024de93e518c74.tar.gz
chromium_src-0f97763f80b69d6bb3fb974c63024de93e518c74.tar.bz2
Roll src/third_party/icu 8ac906f:d8b2a9d7 (svn 292317:292476)
Summary of changes available at: https://chromium.googlesource.com/chromium/deps/icu52/+log/8ac906f..6ea11f3 https://chromium.googlesource.com/chromium/deps/icu52/+log/6ea11f3..d8b2a9d In addition, change the encoding menu, encoding preferences, and browser tests accordingly; 1. unify iso-8859-1 with windows-1252 in the menu (the name to use is windows-1252) 2. update the encoding preference (default encoding, etc) for various locales. 3. update the test result/expectation. Re-enable tests disabled in https://codereview.chromium.org/643993004/ TBR=jsbell@chromium.org BUG=412053 TEST=browser_tests BrowserEncodingTest.TestEncodingAutoDetect BrowserEncodingTest.TestOverrideEncoding EncodingAliases/BrowserEncodingTest.TestEncodingAliasMapping SavePageBrowserTest.FileNameFromPageTitle SavePageBrowserTest.SaveCompleteHTML PrefsTabHelperBrowserTest.WebPrefs TEST=net_unittest --gtest_filter="*ilenameUtil*" TEST=base_unittests --gtest_filter="*Conv*" TEST=Blink: fast/encoding/* TEST=http://www.w3.org/International/tests/repository/encoding/indexes/results-indexes TEST=http://www.w3.org/International/tests/repository/encoding/indexes/results-aliases TEST=http://www.w3.org/International/tests/repository/run?manifest=encoding/indexes&test=windows-1253_test TEST=http://www.w3.org/International/tests/repository/run?manifest=encoding/indexes&test=windows-874_test Review URL: https://codereview.chromium.org/649413002 Cr-Commit-Position: refs/heads/master@{#300643}
Diffstat (limited to 'chrome/browser/browser_encoding_browsertest.cc')
-rw-r--r--chrome/browser/browser_encoding_browsertest.cc17
1 files changed, 3 insertions, 14 deletions
diff --git a/chrome/browser/browser_encoding_browsertest.cc b/chrome/browser/browser_encoding_browsertest.cc
index b26d08f..f4252f9 100644
--- a/chrome/browser/browser_encoding_browsertest.cc
+++ b/chrome/browser/browser_encoding_browsertest.cc
@@ -36,11 +36,7 @@ const EncodingTestData kEncodingTestDatas[] = {
{ "Big5.html", "Big5" },
{ "EUC-JP.html", "EUC-JP" },
{ "gb18030.html", "gb18030" },
-#if 0
- // Disable temporarily until Blink CL
- // (https://codereview.chromium.org/655083002/) is relanded.
{ "iso-8859-1.html", "windows-1252" },
-#endif
{ "ISO-8859-2.html", "ISO-8859-2" },
{ "ISO-8859-4.html", "ISO-8859-4" },
{ "ISO-8859-5.html", "ISO-8859-5" },
@@ -53,10 +49,7 @@ const EncodingTestData kEncodingTestDatas[] = {
{ "KOI8-U.html", "KOI8-U" },
{ "macintosh.html", "macintosh" },
{ "Shift-JIS.html", "Shift_JIS" },
-#if 0
- // See the above.
{ "US-ASCII.html", "windows-1252" }, // http://crbug.com/15801
-#endif
{ "UTF-8.html", "UTF-8" },
{ "UTF-16LE.html", "UTF-16LE" },
{ "windows-874.html", "windows-874" },
@@ -139,7 +132,9 @@ class BrowserEncodingTest
base::FilePath expected_file_name = ui_test_utils::GetTestFilePath(
base::FilePath(kTestDir), expected);
- EXPECT_TRUE(base::ContentsEqual(full_file_name, expected_file_name));
+ EXPECT_TRUE(base::ContentsEqual(full_file_name, expected_file_name)) <<
+ "generated_file = " << full_file_name.AsUTF8Unsafe() <<
+ ", expected_file = " << expected_file_name.AsUTF8Unsafe();
}
void SetUpOnMainThread() override {
@@ -196,10 +191,7 @@ IN_PROC_BROWSER_TEST_F(BrowserEncodingTest, TestOverrideEncoding) {
ui_test_utils::NavigateToURL(browser(), url);
content::WebContents* web_contents =
browser()->tab_strip_model()->GetActiveWebContents();
-#if 0
- // Temporarily disable until the Blink CL to use windows-1252 is relanded.
EXPECT_EQ("windows-1252", web_contents->GetEncoding());
-#endif
// Override the encoding to "gb18030".
const std::string selected_encoding =
@@ -251,12 +243,9 @@ IN_PROC_BROWSER_TEST_F(BrowserEncodingTest, MAYBE_TestEncodingAutoDetect) {
{ "gb18030_with_no_encoding_specified.html",
"expected_gb18030_saved_from_no_encoding_specified.html",
"gb18030" },
-#if 0
- // Disable until the Blink CL to use 'windows-1252' is relanded.
{ "iso-8859-1_with_no_encoding_specified.html",
"expected_iso-8859-1_saved_from_no_encoding_specified.html",
"windows-1252" },
-#endif
{ "ISO-8859-5_with_no_encoding_specified.html",
"expected_ISO-8859-5_saved_from_no_encoding_specified.html",
"ISO-8859-5" },