| OLD | NEW |
| (Empty) |
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | |
| 2 // Use of this source code is governed by a BSD-style license that can be | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 #include "chrome/browser/ui/elide_url.h" | |
| 6 | |
| 7 #include "base/strings/utf_string_conversions.h" | |
| 8 #include "testing/gtest/include/gtest/gtest.h" | |
| 9 #include "ui/gfx/font_list.h" | |
| 10 #include "ui/gfx/text_elider.h" | |
| 11 #include "ui/gfx/text_utils.h" | |
| 12 #include "url/gurl.h" | |
| 13 | |
| 14 using base::UTF8ToUTF16; | |
| 15 using gfx::GetStringWidthF; | |
| 16 using gfx::kEllipsis; | |
| 17 | |
| 18 namespace { | |
| 19 | |
| 20 struct Testcase { | |
| 21 const std::string input; | |
| 22 const std::string output; | |
| 23 }; | |
| 24 | |
| 25 void RunUrlTest(Testcase* testcases, size_t num_testcases) { | |
| 26 static const gfx::FontList font_list; | |
| 27 for (size_t i = 0; i < num_testcases; ++i) { | |
| 28 const GURL url(testcases[i].input); | |
| 29 // Should we test with non-empty language list? | |
| 30 // That's kinda redundant with net_util_unittests. | |
| 31 const float available_width = | |
| 32 GetStringWidthF(UTF8ToUTF16(testcases[i].output), font_list); | |
| 33 EXPECT_EQ(UTF8ToUTF16(testcases[i].output), | |
| 34 ElideUrl(url, font_list, available_width, std::string())); | |
| 35 } | |
| 36 } | |
| 37 | |
| 38 // Test eliding of commonplace URLs. | |
| 39 TEST(TextEliderTest, TestGeneralEliding) { | |
| 40 const std::string kEllipsisStr(kEllipsis); | |
| 41 Testcase testcases[] = { | |
| 42 {"http://www.google.com/intl/en/ads/", | |
| 43 "www.google.com/intl/en/ads/"}, | |
| 44 {"http://www.google.com/intl/en/ads/", "www.google.com/intl/en/ads/"}, | |
| 45 {"http://www.google.com/intl/en/ads/", | |
| 46 "google.com/intl/" + kEllipsisStr + "/ads/"}, | |
| 47 {"http://www.google.com/intl/en/ads/", | |
| 48 "google.com/" + kEllipsisStr + "/ads/"}, | |
| 49 {"http://www.google.com/intl/en/ads/", "google.com/" + kEllipsisStr}, | |
| 50 {"http://www.google.com/intl/en/ads/", "goog" + kEllipsisStr}, | |
| 51 {"https://subdomain.foo.com/bar/filename.html", | |
| 52 "subdomain.foo.com/bar/filename.html"}, | |
| 53 {"https://subdomain.foo.com/bar/filename.html", | |
| 54 "subdomain.foo.com/" + kEllipsisStr + "/filename.html"}, | |
| 55 {"http://subdomain.foo.com/bar/filename.html", | |
| 56 kEllipsisStr + "foo.com/" + kEllipsisStr + "/filename.html"}, | |
| 57 {"http://www.google.com/intl/en/ads/?aLongQueryWhichIsNotRequired", | |
| 58 "www.google.com/intl/en/ads/?aLongQ" + kEllipsisStr}, | |
| 59 }; | |
| 60 | |
| 61 RunUrlTest(testcases, arraysize(testcases)); | |
| 62 } | |
| 63 | |
| 64 // When there is very little space available, the elision code will shorten | |
| 65 // both path AND file name to an ellipsis - ".../...". To avoid this result, | |
| 66 // there is a hack in place that simply treats them as one string in this | |
| 67 // case. | |
| 68 TEST(TextEliderTest, TestTrailingEllipsisSlashEllipsisHack) { | |
| 69 const std::string kEllipsisStr(kEllipsis); | |
| 70 | |
| 71 // Very little space, would cause double ellipsis. | |
| 72 gfx::FontList font_list; | |
| 73 GURL url("http://battersbox.com/directory/foo/peter_paul_and_mary.html"); | |
| 74 float available_width = GetStringWidthF( | |
| 75 UTF8ToUTF16("battersbox.com/" + kEllipsisStr + "/" + kEllipsisStr), | |
| 76 font_list); | |
| 77 | |
| 78 // Create the expected string, after elision. Depending on font size, the | |
| 79 // directory might become /dir... or /di... or/d... - it never should be | |
| 80 // shorter than that. (If it is, the font considers d... to be longer | |
| 81 // than .../... - that should never happen). | |
| 82 ASSERT_GT(GetStringWidthF(UTF8ToUTF16(kEllipsisStr + "/" + kEllipsisStr), | |
| 83 font_list), | |
| 84 GetStringWidthF(UTF8ToUTF16("d" + kEllipsisStr), font_list)); | |
| 85 GURL long_url("http://battersbox.com/directorynameisreallylongtoforcetrunc"); | |
| 86 base::string16 expected = | |
| 87 ElideUrl(long_url, font_list, available_width, std::string()); | |
| 88 // Ensure that the expected result still contains part of the directory name. | |
| 89 ASSERT_GT(expected.length(), std::string("battersbox.com/d").length()); | |
| 90 EXPECT_EQ(expected, | |
| 91 ElideUrl(url, font_list, available_width, std::string())); | |
| 92 | |
| 93 // More space available - elide directories, partially elide filename. | |
| 94 Testcase testcases[] = { | |
| 95 {"http://battersbox.com/directory/foo/peter_paul_and_mary.html", | |
| 96 "battersbox.com/" + kEllipsisStr + "/peter" + kEllipsisStr}, | |
| 97 }; | |
| 98 RunUrlTest(testcases, arraysize(testcases)); | |
| 99 } | |
| 100 | |
| 101 // Test eliding of empty strings, URLs with ports, passwords, queries, etc. | |
| 102 TEST(TextEliderTest, TestMoreEliding) { | |
| 103 const std::string kEllipsisStr(kEllipsis); | |
| 104 Testcase testcases[] = { | |
| 105 {"http://www.google.com/foo?bar", "www.google.com/foo?bar"}, | |
| 106 {"http://xyz.google.com/foo?bar", "xyz.google.com/foo?" + kEllipsisStr}, | |
| 107 {"http://xyz.google.com/foo?bar", "xyz.google.com/foo" + kEllipsisStr}, | |
| 108 {"http://xyz.google.com/foo?bar", "xyz.google.com/fo" + kEllipsisStr}, | |
| 109 {"http://a.b.com/pathname/c?d", "a.b.com/" + kEllipsisStr + "/c?d"}, | |
| 110 {"", ""}, | |
| 111 {"http://foo.bar..example.com...hello/test/filename.html", | |
| 112 "foo.bar..example.com...hello/" + kEllipsisStr + "/filename.html"}, | |
| 113 {"http://foo.bar../", "foo.bar.."}, | |
| 114 {"http://xn--1lq90i.cn/foo", "\xe5\x8c\x97\xe4\xba\xac.cn/foo"}, | |
| 115 {"http://me:mypass@secrethost.com:99/foo?bar#baz", | |
| 116 "secrethost.com:99/foo?bar#baz"}, | |
| 117 {"http://me:mypass@ss%xxfdsf.com/foo", "ss%25xxfdsf.com/foo"}, | |
| 118 {"mailto:elgoato@elgoato.com", "mailto:elgoato@elgoato.com"}, | |
| 119 {"javascript:click(0)", "javascript:click(0)"}, | |
| 120 {"https://chess.eecs.berkeley.edu:4430/login/arbitfilename", | |
| 121 "chess.eecs.berkeley.edu:4430/login/arbitfilename"}, | |
| 122 {"https://chess.eecs.berkeley.edu:4430/login/arbitfilename", | |
| 123 kEllipsisStr + "berkeley.edu:4430/" + kEllipsisStr + "/arbitfilename"}, | |
| 124 | |
| 125 // Unescaping. | |
| 126 {"http://www/%E4%BD%A0%E5%A5%BD?q=%E4%BD%A0%E5%A5%BD#\xe4\xbd\xa0", | |
| 127 "www/\xe4\xbd\xa0\xe5\xa5\xbd?q=\xe4\xbd\xa0\xe5\xa5\xbd#\xe4\xbd\xa0"}, | |
| 128 | |
| 129 // Invalid unescaping for path. The ref will always be valid UTF-8. We don't | |
| 130 // bother to do too many edge cases, since these are handled by the escaper | |
| 131 // unittest. | |
| 132 {"http://www/%E4%A0%E5%A5%BD?q=%E4%BD%A0%E5%A5%BD#\xe4\xbd\xa0", | |
| 133 "www/%E4%A0%E5%A5%BD?q=\xe4\xbd\xa0\xe5\xa5\xbd#\xe4\xbd\xa0"}, | |
| 134 }; | |
| 135 | |
| 136 RunUrlTest(testcases, arraysize(testcases)); | |
| 137 } | |
| 138 | |
| 139 // Test eliding of file: URLs. | |
| 140 TEST(TextEliderTest, TestFileURLEliding) { | |
| 141 const std::string kEllipsisStr(kEllipsis); | |
| 142 Testcase testcases[] = { | |
| 143 {"file:///C:/path1/path2/path3/filename", | |
| 144 "file:///C:/path1/path2/path3/filename"}, | |
| 145 {"file:///C:/path1/path2/path3/filename", | |
| 146 "C:/path1/path2/path3/filename"}, | |
| 147 // GURL parses "file:///C:path" differently on windows than it does on posix. | |
| 148 #if defined(OS_WIN) | |
| 149 {"file:///C:path1/path2/path3/filename", | |
| 150 "C:/path1/path2/" + kEllipsisStr + "/filename"}, | |
| 151 {"file:///C:path1/path2/path3/filename", | |
| 152 "C:/path1/" + kEllipsisStr + "/filename"}, | |
| 153 {"file:///C:path1/path2/path3/filename", | |
| 154 "C:/" + kEllipsisStr + "/filename"}, | |
| 155 #endif | |
| 156 {"file://filer/foo/bar/file", "filer/foo/bar/file"}, | |
| 157 {"file://filer/foo/bar/file", "filer/foo/" + kEllipsisStr + "/file"}, | |
| 158 {"file://filer/foo/bar/file", "filer/" + kEllipsisStr + "/file"}, | |
| 159 }; | |
| 160 | |
| 161 RunUrlTest(testcases, arraysize(testcases)); | |
| 162 } | |
| 163 | |
| 164 TEST(TextEliderTest, TestHostEliding) { | |
| 165 const std::string kEllipsisStr(kEllipsis); | |
| 166 Testcase testcases[] = { | |
| 167 {"http://google.com", "google.com"}, | |
| 168 {"http://subdomain.google.com", kEllipsisStr + ".google.com"}, | |
| 169 {"http://reallyreallyreallylongdomainname.com", | |
| 170 "reallyreallyreallylongdomainname.com"}, | |
| 171 {"http://a.b.c.d.e.f.com", kEllipsisStr + "f.com"}, | |
| 172 {"http://foo", "foo"}, | |
| 173 {"http://foo.bar", "foo.bar"}, | |
| 174 {"http://subdomain.foo.bar", kEllipsisStr + "in.foo.bar"}, | |
| 175 // IOS width calculations are off by a letter from other platforms for | |
| 176 // some strings from other platforms, probably for strings with too | |
| 177 // many kerned letters on the default font set. | |
| 178 #if !defined(OS_IOS) | |
| 179 {"http://subdomain.reallylongdomainname.com", | |
| 180 kEllipsisStr + "ain.reallylongdomainname.com"}, | |
| 181 {"http://a.b.c.d.e.f.com", kEllipsisStr + ".e.f.com"}, | |
| 182 #endif | |
| 183 }; | |
| 184 | |
| 185 for (size_t i = 0; i < arraysize(testcases); ++i) { | |
| 186 const float available_width = | |
| 187 GetStringWidthF(UTF8ToUTF16(testcases[i].output), gfx::FontList()); | |
| 188 EXPECT_EQ(UTF8ToUTF16(testcases[i].output), ElideHost( | |
| 189 GURL(testcases[i].input), gfx::FontList(), available_width)); | |
| 190 } | |
| 191 | |
| 192 // Trying to elide to a really short length will still keep the full TLD+1 | |
| 193 EXPECT_EQ(base::ASCIIToUTF16("google.com"), | |
| 194 ElideHost(GURL("http://google.com"), gfx::FontList(), 2)); | |
| 195 EXPECT_EQ(base::UTF8ToUTF16(kEllipsisStr + ".google.com"), | |
| 196 ElideHost(GURL("http://subdomain.google.com"), gfx::FontList(), 2)); | |
| 197 EXPECT_EQ(base::ASCIIToUTF16("foo.bar"), | |
| 198 ElideHost(GURL("http://foo.bar"), gfx::FontList(), 2)); | |
| 199 } | |
| 200 | |
| 201 } // namespace | |
| OLD | NEW |