Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6971)

Unified Diff: chrome/common/gfx/text_elider_unittest.cc

Issue 113143: Move color_utils, text_elider, drag_utils, accessibility_types, standard_layo... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/common/gfx/text_elider.cc ('k') | chrome/renderer/render_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/gfx/text_elider_unittest.cc
===================================================================
--- chrome/common/gfx/text_elider_unittest.cc (revision 15683)
+++ chrome/common/gfx/text_elider_unittest.cc (working copy)
@@ -1,267 +0,0 @@
-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "app/gfx/chrome_font.h"
-#include "base/file_path.h"
-#include "base/string_util.h"
-#include "chrome/common/gfx/text_elider.h"
-#include "googleurl/src/gurl.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-using namespace gfx;
-
-namespace {
-
-const wchar_t kEllipsis[] = L"\x2026";
-
-struct Testcase {
- const std::string input;
- const std::wstring output;
-};
-
-struct FileTestcase {
- const FilePath::StringType input;
- const std::wstring output;
-};
-
-struct WideTestcase {
- const std::wstring input;
- const std::wstring output;
-};
-
-struct TestData {
- const std::string a;
- const std::string b;
- const int compare_result;
-};
-
-void RunTest(Testcase* testcases, size_t num_testcases) {
- static const ChromeFont font;
- 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.
- EXPECT_EQ(testcases[i].output,
- ElideUrl(url, font, font.GetStringWidth(testcases[i].output),
- std::wstring()));
- }
-}
-
-} // namespace
-
-// Test eliding of commonplace URLs.
-TEST(TextEliderTest, TestGeneralEliding) {
- const std::wstring kEllipsisStr(kEllipsis);
- Testcase testcases[] = {
- {"http://www.google.com/intl/en/ads/",
- L"http://www.google.com/intl/en/ads/"},
- {"http://www.google.com/intl/en/ads/", L"www.google.com/intl/en/ads/"},
-// TODO(port): make this test case work on mac.
-#if !defined(OS_MACOSX)
- {"http://www.google.com/intl/en/ads/",
- L"google.com/intl/" + kEllipsisStr + L"/ads/"},
-#endif
- {"http://www.google.com/intl/en/ads/",
- L"google.com/" + kEllipsisStr + L"/ads/"},
- {"http://www.google.com/intl/en/ads/", L"google.com/" + kEllipsisStr},
- {"http://www.google.com/intl/en/ads/", L"goog" + kEllipsisStr},
- {"https://subdomain.foo.com/bar/filename.html",
- L"subdomain.foo.com/bar/filename.html"},
- {"https://subdomain.foo.com/bar/filename.html",
- L"subdomain.foo.com/" + kEllipsisStr + L"/filename.html"},
- {"http://subdomain.foo.com/bar/filename.html",
- kEllipsisStr + L"foo.com/" + kEllipsisStr + L"/filename.html"},
- {"http://www.google.com/intl/en/ads/?aLongQueryWhichIsNotRequired",
- L"http://www.google.com/intl/en/ads/?aLongQ" + kEllipsisStr},
- };
-
- RunTest(testcases, arraysize(testcases));
-}
-
-// Test eliding of empty strings, URLs with ports, passwords, queries, etc.
-TEST(TextEliderTest, TestMoreEliding) {
- const std::wstring kEllipsisStr(kEllipsis);
- Testcase testcases[] = {
- {"http://www.google.com/foo?bar", L"http://www.google.com/foo?bar"},
- {"http://xyz.google.com/foo?bar", L"xyz.google.com/foo?" + kEllipsisStr},
- {"http://xyz.google.com/foo?bar", L"xyz.google.com/foo" + kEllipsisStr},
- {"http://xyz.google.com/foo?bar", L"xyz.google.com/fo" + kEllipsisStr},
- {"http://a.b.com/pathname/c?d", L"a.b.com/" + kEllipsisStr + L"/c?d"},
- {"", L""},
- {"http://foo.bar..example.com...hello/test/filename.html",
- L"foo.bar..example.com...hello/" + kEllipsisStr + L"/filename.html"},
- {"http://foo.bar../", L"http://foo.bar../"},
- {"http://xn--1lq90i.cn/foo", L"http://\x5317\x4eac.cn/foo"},
- {"http://me:mypass@secrethost.com:99/foo?bar#baz",
- L"http://secrethost.com:99/foo?bar#baz"},
- {"http://me:mypass@ss%xxfdsf.com/foo", L"http://ss%25xxfdsf.com/foo"},
- {"mailto:elgoato@elgoato.com", L"mailto:elgoato@elgoato.com"},
- {"javascript:click(0)", L"javascript:click(0)"},
- {"https://chess.eecs.berkeley.edu:4430/login/arbitfilename",
- L"chess.eecs.berkeley.edu:4430/login/arbitfilename"},
- {"https://chess.eecs.berkeley.edu:4430/login/arbitfilename",
- kEllipsisStr + L"berkeley.edu:4430/" + kEllipsisStr + L"/arbitfilename"},
-
- // Unescaping.
- {"http://www/%E4%BD%A0%E5%A5%BD?q=%E4%BD%A0%E5%A5%BD#\xe4\xbd\xa0",
- L"http://www/\x4f60\x597d?q=\x4f60\x597d#\x4f60"},
-
- // Invalid unescaping for path. The ref will always be valid UTF-8. We don't
- // bother to do too many edge cases, since these are handled by the escaper
- // unittest.
- {"http://www/%E4%A0%E5%A5%BD?q=%E4%BD%A0%E5%A5%BD#\xe4\xbd\xa0",
- L"http://www/%E4%A0%E5%A5%BD?q=\x4f60\x597d#\x4f60"},
- };
-
- RunTest(testcases, arraysize(testcases));
-}
-
-// Test eliding of file: URLs.
-TEST(TextEliderTest, TestFileURLEliding) {
- const std::wstring kEllipsisStr(kEllipsis);
- Testcase testcases[] = {
- {"file:///C:/path1/path2/path3/filename",
- L"file:///C:/path1/path2/path3/filename"},
- {"file:///C:/path1/path2/path3/filename",
- L"C:/path1/path2/path3/filename"},
-// GURL parses "file:///C:path" differently on windows than it does on posix.
-#if defined(OS_WIN)
- {"file:///C:path1/path2/path3/filename",
- L"C:/path1/path2/" + kEllipsisStr + L"/filename"},
- {"file:///C:path1/path2/path3/filename",
- L"C:/path1/" + kEllipsisStr + L"/filename"},
- {"file:///C:path1/path2/path3/filename",
- L"C:/" + kEllipsisStr + L"/filename"},
-#endif
- {"file://filer/foo/bar/file", L"filer/foo/bar/file"},
- {"file://filer/foo/bar/file", L"filer/foo/" + kEllipsisStr + L"/file"},
- {"file://filer/foo/bar/file", L"filer/" + kEllipsisStr + L"/file"},
- };
-
- RunTest(testcases, arraysize(testcases));
-}
-
-TEST(TextEliderTest, TestFilenameEliding) {
- const std::wstring kEllipsisStr(kEllipsis);
- const FilePath::StringType kPathSeparator =
- FilePath::StringType().append(1, FilePath::kSeparators[0]);
-
- FileTestcase testcases[] = {
- {FILE_PATH_LITERAL(""), L""},
- {FILE_PATH_LITERAL("."), L"."},
- {FILE_PATH_LITERAL("filename.exe"), L"filename.exe"},
- {FILE_PATH_LITERAL(".longext"), L".longext"},
- {FILE_PATH_LITERAL("pie"), L"pie"},
- {FILE_PATH_LITERAL("c:") + kPathSeparator + FILE_PATH_LITERAL("path") +
- kPathSeparator + FILE_PATH_LITERAL("filename.pie"),
- L"filename.pie"},
- {FILE_PATH_LITERAL("c:") + kPathSeparator + FILE_PATH_LITERAL("path") +
- kPathSeparator + FILE_PATH_LITERAL("longfilename.pie"),
- L"long" + kEllipsisStr + L".pie"},
- {FILE_PATH_LITERAL("http://path.com/filename.pie"), L"filename.pie"},
- {FILE_PATH_LITERAL("http://path.com/longfilename.pie"),
- L"long" + kEllipsisStr + L".pie"},
- {FILE_PATH_LITERAL("piesmashingtacularpants"), L"pie" + kEllipsisStr},
- {FILE_PATH_LITERAL(".piesmashingtacularpants"), L".pie" + kEllipsisStr},
- {FILE_PATH_LITERAL("cheese."), L"cheese."},
- {FILE_PATH_LITERAL("file name.longext"),
- L"file" + kEllipsisStr + L".longext"},
- {FILE_PATH_LITERAL("fil ename.longext"),
- L"fil " + kEllipsisStr + L".longext"},
- {FILE_PATH_LITERAL("filename.longext"),
- L"file" + kEllipsisStr + L".longext"},
- {FILE_PATH_LITERAL("filename.middleext.longext"),
- L"filename.mid" + kEllipsisStr + L".longext"}
- };
-
- static const ChromeFont font;
- for (size_t i = 0; i < arraysize(testcases); ++i) {
- FilePath filepath(testcases[i].input);
- EXPECT_EQ(testcases[i].output, ElideFilename(filepath,
- font,
- font.GetStringWidth(testcases[i].output)));
- }
-}
-
-TEST(TextEliderTest, ElideTextLongStrings) {
- const std::wstring kEllipsisStr(kEllipsis);
- std::wstring data_scheme(L"data:text/plain,");
-
- std::wstring ten_a(10, L'a');
- std::wstring hundred_a(100, L'a');
- std::wstring thousand_a(1000, L'a');
- std::wstring ten_thousand_a(10000, L'a');
- std::wstring hundred_thousand_a(100000, L'a');
- std::wstring million_a(1000000, L'a');
-
- WideTestcase testcases[] = {
- {data_scheme + ten_a,
- data_scheme + ten_a},
- {data_scheme + hundred_a,
- data_scheme + hundred_a},
- {data_scheme + thousand_a,
- data_scheme + std::wstring(156, L'a') + kEllipsisStr},
- {data_scheme + ten_thousand_a,
- data_scheme + std::wstring(156, L'a') + kEllipsisStr},
- {data_scheme + hundred_thousand_a,
- data_scheme + std::wstring(156, L'a') + kEllipsisStr},
- {data_scheme + million_a,
- data_scheme + std::wstring(156, L'a') + kEllipsisStr},
- };
-
- const ChromeFont font;
- int ellipsis_width = font.GetStringWidth(kEllipsisStr);
- for (size_t i = 0; i < arraysize(testcases); ++i) {
- // Compare sizes rather than actual contents because if the test fails,
- // output is rather long.
- EXPECT_EQ(testcases[i].output.size(),
- ElideText(testcases[i].input, font,
- font.GetStringWidth(testcases[i].output)).size());
- EXPECT_EQ(kEllipsisStr,
- ElideText(testcases[i].input, font, ellipsis_width));
- }
-}
-
-// Verifies display_url is set correctly.
-TEST(TextEliderTest, SortedDisplayURL) {
- gfx::SortedDisplayURL d_url(GURL("http://www.google.com/"), std::wstring());
- EXPECT_EQ("http://www.google.com/", UTF16ToASCII(d_url.display_url()));
-}
-
-// Verifies DisplayURL::Compare works correctly.
-TEST(TextEliderTest, SortedDisplayURLCompare) {
- UErrorCode create_status = U_ZERO_ERROR;
- scoped_ptr<Collator> collator(Collator::createInstance(create_status));
- if (!U_SUCCESS(create_status))
- return;
-
- TestData tests[] = {
- // IDN comparison. Hosts equal, so compares on path.
- { "http://xn--1lq90i.cn/a", "http://xn--1lq90i.cn/b", -1},
-
- // Because the host and after host match, this compares the full url.
- { "http://www.x/b", "http://x/b", -1 },
-
- // Because the host and after host match, this compares the full url.
- { "http://www.a:1/b", "http://a:1/b", 1 },
-
- // The hosts match, so these end up comparing on the after host portion.
- { "http://www.x:0/b", "http://x:1/b", -1 },
- { "http://www.x/a", "http://x/b", -1 },
- { "http://x/b", "http://www.x/a", 1 },
-
- // Trivial Equality.
- { "http://a/", "http://a/", 0 },
-
- // Compares just hosts.
- { "http://www.a/", "http://b/", -1 },
- };
-
- for (size_t i = 0; i < arraysize(tests); ++i) {
- gfx::SortedDisplayURL url1(GURL(tests[i].a), std::wstring());
- gfx::SortedDisplayURL url2(GURL(tests[i].b), std::wstring());
- EXPECT_EQ(tests[i].compare_result, url1.Compare(url2, collator.get()));
- EXPECT_EQ(-tests[i].compare_result, url2.Compare(url1, collator.get()));
- }
-}
« no previous file with comments | « chrome/common/gfx/text_elider.cc ('k') | chrome/renderer/render_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698