diff options
author | jshin <jshin@chromium.org> | 2014-10-21 23:51:29 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-22 06:52:48 +0000 |
commit | 0f97763f80b69d6bb3fb974c63024de93e518c74 (patch) | |
tree | 0a1a99286f8d6211611a7c1c19550abcbb31b832 /chrome/browser/browser_encoding_browsertest.cc | |
parent | 161168015de91abe74ba7be393a12a63f99a0636 (diff) | |
download | chromium_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.cc | 17 |
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" }, |