Index: components/url_formatter/elide_url_unittest.cc |
diff --git a/components/url_formatter/elide_url_unittest.cc b/components/url_formatter/elide_url_unittest.cc |
index ede2bf6eea7eaaf05d20b05bc147bc520b06590a..1d911a8e216eb55438fb464eea1791380477052e 100644 |
--- a/components/url_formatter/elide_url_unittest.cc |
+++ b/components/url_formatter/elide_url_unittest.cc |
@@ -36,13 +36,10 @@ void RunUrlTest(Testcase* testcases, size_t num_testcases) { |
static const gfx::FontList font_list; |
for (size_t i = 0; i < num_testcases; ++i) { |
const GURL url(testcases[i].input); |
- // Should we test with non-empty language list? |
- // That's kinda redundant with net_util_unittests. |
const float available_width = |
gfx::GetStringWidthF(base::UTF8ToUTF16(testcases[i].output), font_list); |
EXPECT_EQ(base::UTF8ToUTF16(testcases[i].output), |
- url_formatter::ElideUrl(url, font_list, available_width, |
- std::string())); |
+ url_formatter::ElideUrl(url, font_list, available_width)); |
} |
} |
@@ -95,11 +92,10 @@ TEST(TextEliderTest, TestTrailingEllipsisSlashEllipsisHack) { |
gfx::GetStringWidthF(base::UTF8ToUTF16("d" + kEllipsisStr), font_list)); |
GURL long_url("http://battersbox.com/directorynameisreallylongtoforcetrunc"); |
base::string16 expected = url_formatter::ElideUrl( |
- long_url, font_list, available_width, std::string()); |
+ long_url, font_list, available_width); |
// Ensure that the expected result still contains part of the directory name. |
ASSERT_GT(expected.length(), std::string("battersbox.com/d").length()); |
- EXPECT_EQ(expected, url_formatter::ElideUrl(url, font_list, available_width, |
- std::string())); |
+ EXPECT_EQ(expected, url_formatter::ElideUrl(url, font_list, available_width)); |
// More space available - elide directories, partially elide filename. |
Testcase testcases[] = { |
@@ -327,42 +323,27 @@ TEST(TextEliderTest, FormatUrlForSecurityDisplay) { |
L"blob:http%3A//www.html5rocks.com/" |
L"4d4ff040-6d61-4446-86d3-13ca07ec9ab9"}}; |
- const char languages[] = "zh-TW,en-US,en,am,ar-EG,ar"; |
for (size_t i = 0; i < arraysize(tests); ++i) { |
- base::string16 formatted = url_formatter::FormatUrlForSecurityDisplay( |
- GURL(tests[i].input), std::string()); |
+ base::string16 formatted = |
+ url_formatter::FormatUrlForSecurityDisplay(GURL(tests[i].input)); |
EXPECT_EQ(base::WideToUTF16(tests[i].output), formatted) |
<< tests[i].description; |
base::string16 formatted_omit_scheme = |
url_formatter::FormatUrlForSecurityDisplayOmitScheme( |
- GURL(tests[i].input), std::string()); |
+ GURL(tests[i].input)); |
EXPECT_EQ(base::WideToUTF16(tests[i].output_omit_scheme), |
formatted_omit_scheme) |
<< tests[i].description; |
- |
- base::string16 formatted_with_languages = |
- url_formatter::FormatUrlForSecurityDisplay(GURL(tests[i].input), |
- languages); |
- EXPECT_EQ(base::WideToUTF16(tests[i].output), formatted_with_languages) |
- << tests[i].description; |
- |
- base::string16 formatted_with_languages_omit_scheme = |
- url_formatter::FormatUrlForSecurityDisplayOmitScheme( |
- GURL(tests[i].input), languages); |
- EXPECT_EQ(base::WideToUTF16(tests[i].output_omit_scheme), |
- formatted_with_languages_omit_scheme) |
- << tests[i].description; |
} |
base::string16 formatted = |
- url_formatter::FormatUrlForSecurityDisplay(GURL(), std::string()); |
+ url_formatter::FormatUrlForSecurityDisplay(GURL()); |
EXPECT_EQ(base::string16(), formatted) |
<< "Explicitly test the 0-argument GURL constructor"; |
base::string16 formatted_omit_scheme = |
- url_formatter::FormatUrlForSecurityDisplayOmitScheme(GURL(), |
- std::string()); |
+ url_formatter::FormatUrlForSecurityDisplayOmitScheme(GURL()); |
EXPECT_EQ(base::string16(), formatted_omit_scheme) |
<< "Explicitly test the 0-argument GURL constructor"; |
} |