diff options
author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-09 22:15:25 +0000 |
---|---|---|
committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-09 22:15:25 +0000 |
commit | dbb97ba9f0608c2ebf6f02f750df9c5e433b770e (patch) | |
tree | 6da1f4050039906a020366d937107190c86b6776 /ui | |
parent | 9045c60ac4a8cbeeb911aadcd771fb4ec020703c (diff) | |
download | chromium_src-dbb97ba9f0608c2ebf6f02f750df9c5e433b770e.zip chromium_src-dbb97ba9f0608c2ebf6f02f750df9c5e433b770e.tar.gz chromium_src-dbb97ba9f0608c2ebf6f02f750df9c5e433b770e.tar.bz2 |
Move text_elider to gfx.
R=sky@chromium.org
http://crbug.com/103304
Review URL: https://codereview.chromium.org/23731010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222106 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui')
-rw-r--r-- | ui/base/cocoa/menu_controller.mm | 4 | ||||
-rw-r--r-- | ui/base/text/text_elider.cc | 28 | ||||
-rw-r--r-- | ui/base/text/text_elider_unittest.cc | 2 | ||||
-rw-r--r-- | ui/gfx/canvas_skia.cc | 20 | ||||
-rw-r--r-- | ui/gfx/render_text.cc | 4 | ||||
-rw-r--r-- | ui/gfx/text_elider.cc | 1172 | ||||
-rw-r--r-- | ui/gfx/text_elider.h | 232 | ||||
-rw-r--r-- | ui/gfx/text_elider_unittest.cc | 934 | ||||
-rw-r--r-- | ui/message_center/cocoa/notification_controller.mm | 12 | ||||
-rw-r--r-- | ui/message_center/views/bounded_label.cc | 12 | ||||
-rw-r--r-- | ui/message_center/views/notification_view.cc | 8 | ||||
-rw-r--r-- | ui/ui.gyp | 4 | ||||
-rw-r--r-- | ui/ui_unittests.gypi | 2 | ||||
-rw-r--r-- | ui/views/controls/label.cc | 13 | ||||
-rw-r--r-- | ui/views/controls/styled_label.cc | 6 | ||||
-rw-r--r-- | ui/views/corewm/tooltip_controller.cc | 7 | ||||
-rw-r--r-- | ui/views/corewm/tooltip_controller_unittest.cc | 6 | ||||
-rw-r--r-- | ui/views/widget/tooltip_manager.cc | 4 |
18 files changed, 2405 insertions, 65 deletions
diff --git a/ui/base/cocoa/menu_controller.mm b/ui/base/cocoa/menu_controller.mm index ebec346..c7b8dff 100644 --- a/ui/base/cocoa/menu_controller.mm +++ b/ui/base/cocoa/menu_controller.mm @@ -11,9 +11,9 @@ #import "ui/base/cocoa/cocoa_event_utils.h" #include "ui/base/l10n/l10n_util_mac.h" #include "ui/base/models/simple_menu_model.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/font.h" #include "ui/gfx/image/image.h" +#include "ui/gfx/text_elider.h" @interface MenuController (Private) - (void)addSeparatorToMenu:(NSMenu*)menu @@ -30,7 +30,7 @@ NSFont* nsfont = [NSFont menuBarFontOfSize:0]; // 0 means "default" gfx::Font font(base::SysNSStringToUTF8([nsfont fontName]), static_cast<int>([nsfont pointSize])); - return ui::ElideText(title, font, width, ui::ELIDE_AT_END); + return gfx::ElideText(title, font, width, gfx::ELIDE_AT_END); } - (id)init { diff --git a/ui/base/text/text_elider.cc b/ui/base/text/text_elider.cc index 76dbb71..ed7b4d3 100644 --- a/ui/base/text/text_elider.cc +++ b/ui/base/text/text_elider.cc @@ -7,7 +7,7 @@ // Note that several of the functions declared in text_elider.h are implemented // in this file using helper classes in an unnamed namespace. -#include "ui/base/text/text_elider.h" +#include "ui/gfx/text_elider.h" #include <string> #include <vector> @@ -814,7 +814,7 @@ class RectangleText { RectangleText(const gfx::FontList& font_list, int available_pixel_width, int available_pixel_height, - ui::WordWrapBehavior wrap_behavior, + gfx::WordWrapBehavior wrap_behavior, std::vector<string16>* lines) : font_list_(font_list), line_height_(font_list.GetHeight()), @@ -882,7 +882,7 @@ class RectangleText { const int available_pixel_height_; // The wrap behavior for words that are too long to fit on a single line. - const ui::WordWrapBehavior wrap_behavior_; + const gfx::WordWrapBehavior wrap_behavior_; // The current running width. int current_width_; @@ -938,8 +938,8 @@ int RectangleText::Finalize() { } if (last_line_ended_in_lf_) lines_->push_back(string16()); - return (insufficient_width_ ? ui::INSUFFICIENT_SPACE_HORIZONTAL : 0) | - (insufficient_height_ ? ui::INSUFFICIENT_SPACE_VERTICAL : 0); + return (insufficient_width_ ? gfx::INSUFFICIENT_SPACE_HORIZONTAL : 0) | + (insufficient_height_ ? gfx::INSUFFICIENT_SPACE_VERTICAL : 0); } void RectangleText::AddLine(const string16& line) { @@ -984,8 +984,8 @@ int RectangleText::WrapWord(const string16& word) { bool first_fragment = true; while (!insufficient_height_ && !text.empty()) { string16 fragment = - ui::ElideText(text, font_list_, available_pixel_width_, - ui::TRUNCATE_AT_END); + gfx::ElideText(text, font_list_, available_pixel_width_, + gfx::TRUNCATE_AT_END); // At least one character has to be added at every line, even if the // available space is too small. if(fragment.empty()) @@ -1009,17 +1009,17 @@ int RectangleText::AddWordOverflow(const string16& word) { lines_added++; } - if (wrap_behavior_ == ui::IGNORE_LONG_WORDS) { + if (wrap_behavior_ == gfx::IGNORE_LONG_WORDS) { current_line_ = word; current_width_ = available_pixel_width_; - } else if (wrap_behavior_ == ui::WRAP_LONG_WORDS) { + } else if (wrap_behavior_ == gfx::WRAP_LONG_WORDS) { lines_added += WrapWord(word); } else { - const ui::ElideBehavior elide_behavior = - (wrap_behavior_ == ui::ELIDE_LONG_WORDS ? ui::ELIDE_AT_END : - ui::TRUNCATE_AT_END); + const gfx::ElideBehavior elide_behavior = + (wrap_behavior_ == gfx::ELIDE_LONG_WORDS ? gfx::ELIDE_AT_END : + gfx::TRUNCATE_AT_END); const string16 elided_word = - ui::ElideText(word, font_list_, available_pixel_width_, elide_behavior); + gfx::ElideText(word, font_list_, available_pixel_width_, elide_behavior); AddToCurrentLine(elided_word); insufficient_width_ = true; } @@ -1039,7 +1039,7 @@ int RectangleText::AddWord(const string16& word) { // Append the non-trimmed word, in case more words are added after. AddToCurrentLine(word); } else { - lines_added = AddWordOverflow(wrap_behavior_ == ui::IGNORE_LONG_WORDS ? + lines_added = AddWordOverflow(wrap_behavior_ == gfx::IGNORE_LONG_WORDS ? trimmed : word); } return lines_added; diff --git a/ui/base/text/text_elider_unittest.cc b/ui/base/text/text_elider_unittest.cc index 3c06cb5..6522c3f 100644 --- a/ui/base/text/text_elider_unittest.cc +++ b/ui/base/text/text_elider_unittest.cc @@ -4,7 +4,7 @@ // // Unit tests for eliding and formatting utility functions. -#include "ui/base/text/text_elider.h" +#include "ui/gfx/text_elider.h" #include "base/files/file_path.h" #include "base/i18n/rtl.h" diff --git a/ui/gfx/canvas_skia.cc b/ui/gfx/canvas_skia.cc index fcbe1e6..5d38606 100644 --- a/ui/gfx/canvas_skia.cc +++ b/ui/gfx/canvas_skia.cc @@ -7,13 +7,13 @@ #include "base/i18n/rtl.h" #include "base/logging.h" #include "base/memory/scoped_ptr.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/font_list.h" #include "ui/gfx/insets.h" #include "ui/gfx/range/range.h" #include "ui/gfx/rect.h" #include "ui/gfx/render_text.h" #include "ui/gfx/shadow_value.h" +#include "ui/gfx/text_elider.h" #include "ui/gfx/text_utils.h" namespace gfx { @@ -103,7 +103,7 @@ void ElideTextAndAdjustRange(const FontList& font_list, gfx::Range* range) { const base::char16 start_char = (range->IsValid() ? text->at(range->start()) : 0); - *text = ui::ElideText(*text, font_list, width, ui::ELIDE_AT_END); + *text = gfx::ElideText(*text, font_list, width, gfx::ELIDE_AT_END); if (!range->IsValid()) return; if (range->start() >= text->length() || @@ -183,15 +183,15 @@ void Canvas::SizeStringInt(const base::string16& text, #endif if ((flags & MULTI_LINE) && *width != 0) { - ui::WordWrapBehavior wrap_behavior = ui::TRUNCATE_LONG_WORDS; + gfx::WordWrapBehavior wrap_behavior = gfx::TRUNCATE_LONG_WORDS; if (flags & CHARACTER_BREAK) - wrap_behavior = ui::WRAP_LONG_WORDS; + wrap_behavior = gfx::WRAP_LONG_WORDS; else if (!(flags & NO_ELLIPSIS)) - wrap_behavior = ui::ELIDE_LONG_WORDS; + wrap_behavior = gfx::ELIDE_LONG_WORDS; Rect rect(*width, INT_MAX); std::vector<base::string16> strings; - ui::ElideRectangleText(adjusted_text, font_list, + gfx::ElideRectangleText(adjusted_text, font_list, rect.width(), rect.height(), wrap_behavior, &strings); scoped_ptr<RenderText> render_text(RenderText::CreateInstance()); @@ -258,14 +258,14 @@ void Canvas::DrawStringRectWithShadows(const base::string16& text, render_text->SetTextShadows(shadows); if (flags & MULTI_LINE) { - ui::WordWrapBehavior wrap_behavior = ui::IGNORE_LONG_WORDS; + gfx::WordWrapBehavior wrap_behavior = gfx::IGNORE_LONG_WORDS; if (flags & CHARACTER_BREAK) - wrap_behavior = ui::WRAP_LONG_WORDS; + wrap_behavior = gfx::WRAP_LONG_WORDS; else if (!(flags & NO_ELLIPSIS)) - wrap_behavior = ui::ELIDE_LONG_WORDS; + wrap_behavior = gfx::ELIDE_LONG_WORDS; std::vector<base::string16> strings; - ui::ElideRectangleText(adjusted_text, + gfx::ElideRectangleText(adjusted_text, font_list, text_bounds.width(), text_bounds.height(), wrap_behavior, diff --git a/ui/gfx/render_text.cc b/ui/gfx/render_text.cc index 25e0027..06640e0 100644 --- a/ui/gfx/render_text.cc +++ b/ui/gfx/render_text.cc @@ -13,12 +13,12 @@ #include "third_party/icu/source/common/unicode/utf16.h" #include "third_party/skia/include/core/SkTypeface.h" #include "third_party/skia/include/effects/SkGradientShader.h" -#include "ui/base/text/text_elider.h" #include "ui/base/text/utf16_indexing.h" #include "ui/gfx/canvas.h" #include "ui/gfx/insets.h" #include "ui/gfx/skia_util.h" #include "ui/gfx/text_constants.h" +#include "ui/gfx/text_elider.h" namespace gfx { @@ -977,7 +977,7 @@ void RenderText::UpdateLayoutText() { // Truncate the text at a valid character break and append an ellipsis. icu::StringCharacterIterator iter(text.c_str()); iter.setIndex32(truncate_length_ - 1); - layout_text_.assign(text.substr(0, iter.getIndex()) + ui::kEllipsisUTF16); + layout_text_.assign(text.substr(0, iter.getIndex()) + gfx::kEllipsisUTF16); } } diff --git a/ui/gfx/text_elider.cc b/ui/gfx/text_elider.cc new file mode 100644 index 0000000..bc4eb3b --- /dev/null +++ b/ui/gfx/text_elider.cc @@ -0,0 +1,1172 @@ +// Copyright (c) 2012 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. +// +// This file implements utility functions for eliding and formatting UI text. +// +// Note that several of the functions declared in text_elider.h are implemented +// in this file using helper classes in an unnamed namespace. + +#include "ui/gfx/text_elider.h" + +#include <string> +#include <vector> + +#include "base/files/file_path.h" +#include "base/i18n/break_iterator.h" +#include "base/i18n/char_iterator.h" +#include "base/i18n/rtl.h" +#include "base/memory/scoped_ptr.h" +#include "base/strings/string_split.h" +#include "base/strings/string_util.h" +#include "base/strings/sys_string_conversions.h" +#include "base/strings/utf_string_conversions.h" +#include "net/base/escape.h" +#include "net/base/net_util.h" +#include "net/base/registry_controlled_domains/registry_controlled_domain.h" +#include "third_party/icu/source/common/unicode/rbbi.h" +#include "third_party/icu/source/common/unicode/uloc.h" +#include "ui/gfx/font_list.h" +#include "ui/gfx/text_utils.h" +#include "url/gurl.h" + +namespace gfx { + +// U+2026 in utf8 +const char kEllipsis[] = "\xE2\x80\xA6"; +const char16 kEllipsisUTF16[] = { 0x2026, 0 }; +const char16 kForwardSlash = '/'; + +namespace { + +// Helper class to split + elide text, while respecting UTF16 surrogate pairs. +class StringSlicer { + public: + StringSlicer(const string16& text, + const string16& ellipsis, + bool elide_in_middle) + : text_(text), + ellipsis_(ellipsis), + elide_in_middle_(elide_in_middle) { + } + + // Cuts |text_| to be |length| characters long. If |elide_in_middle_| is true, + // the middle of the string is removed to leave equal-length pieces from the + // beginning and end of the string; otherwise, the end of the string is + // removed and only the beginning remains. If |insert_ellipsis| is true, + // then an ellipsis character will be inserted at the cut point. + string16 CutString(size_t length, bool insert_ellipsis) { + const string16 ellipsis_text = insert_ellipsis ? ellipsis_ : string16(); + + if (!elide_in_middle_) + return text_.substr(0, FindValidBoundaryBefore(length)) + ellipsis_text; + + // We put the extra character, if any, before the cut. + const size_t half_length = length / 2; + const size_t prefix_length = FindValidBoundaryBefore(length - half_length); + const size_t suffix_start_guess = text_.length() - half_length; + const size_t suffix_start = FindValidBoundaryAfter(suffix_start_guess); + const size_t suffix_length = + half_length - (suffix_start_guess - suffix_start); + return text_.substr(0, prefix_length) + ellipsis_text + + text_.substr(suffix_start, suffix_length); + } + + private: + // Returns a valid cut boundary at or before |index|. + size_t FindValidBoundaryBefore(size_t index) const { + DCHECK_LE(index, text_.length()); + if (index != text_.length()) + U16_SET_CP_START(text_.data(), 0, index); + return index; + } + + // Returns a valid cut boundary at or after |index|. + size_t FindValidBoundaryAfter(size_t index) const { + DCHECK_LE(index, text_.length()); + if (index != text_.length()) + U16_SET_CP_LIMIT(text_.data(), 0, index, text_.length()); + return index; + } + + // The text to be sliced. + const string16& text_; + + // Ellipsis string to use. + const string16& ellipsis_; + + // If true, the middle of the string will be elided. + bool elide_in_middle_; + + DISALLOW_COPY_AND_ASSIGN(StringSlicer); +}; + +// Build a path from the first |num_components| elements in |path_elements|. +// Prepends |path_prefix|, appends |filename|, inserts ellipsis if appropriate. +string16 BuildPathFromComponents(const string16& path_prefix, + const std::vector<string16>& path_elements, + const string16& filename, + size_t num_components) { + // Add the initial elements of the path. + string16 path = path_prefix; + + // Build path from first |num_components| elements. + for (size_t j = 0; j < num_components; ++j) + path += path_elements[j] + kForwardSlash; + + // Add |filename|, ellipsis if necessary. + if (num_components != (path_elements.size() - 1)) + path += string16(kEllipsisUTF16) + kForwardSlash; + path += filename; + + return path; +} + +// Takes a prefix (Domain, or Domain+subdomain) and a collection of path +// components and elides if possible. Returns a string containing the longest +// possible elided path, or an empty string if elision is not possible. +string16 ElideComponentizedPath(const string16& url_path_prefix, + const std::vector<string16>& url_path_elements, + const string16& url_filename, + const string16& url_query, + const gfx::FontList& font_list, + int available_pixel_width) { + const size_t url_path_number_of_elements = url_path_elements.size(); + + CHECK(url_path_number_of_elements); + for (size_t i = url_path_number_of_elements - 1; i > 0; --i) { + string16 elided_path = BuildPathFromComponents(url_path_prefix, + url_path_elements, url_filename, i); + if (available_pixel_width >= gfx::GetStringWidth(elided_path, font_list)) + return ElideText(elided_path + url_query, font_list, + available_pixel_width, ELIDE_AT_END); + } + + return string16(); +} + +} // namespace + +string16 ElideEmail(const string16& email, + const gfx::FontList& font_list, + int available_pixel_width) { + if (gfx::GetStringWidth(email, font_list) <= available_pixel_width) + return email; + + // Split the email into its local-part (username) and domain-part. The email + // spec technically allows for @ symbols in the local-part (username) of the + // email under some special requirements. It is guaranteed that there is no @ + // symbol in the domain part of the email however so splitting at the last @ + // symbol is safe. + const size_t split_index = email.find_last_of('@'); + DCHECK_NE(split_index, string16::npos); + string16 username = email.substr(0, split_index); + string16 domain = email.substr(split_index + 1); + DCHECK(!username.empty()); + DCHECK(!domain.empty()); + + // Subtract the @ symbol from the available width as it is mandatory. + const string16 kAtSignUTF16 = ASCIIToUTF16("@"); + available_pixel_width -= gfx::GetStringWidth(kAtSignUTF16, font_list); + + // Check whether eliding the domain is necessary: if eliding the username + // is sufficient, the domain will not be elided. + const int full_username_width = gfx::GetStringWidth(username, font_list); + const int available_domain_width = + available_pixel_width - + std::min(full_username_width, + gfx::GetStringWidth(username.substr(0, 1) + kEllipsisUTF16, + font_list)); + if (gfx::GetStringWidth(domain, font_list) > available_domain_width) { + // Elide the domain so that it only takes half of the available width. + // Should the username not need all the width available in its half, the + // domain will occupy the leftover width. + // If |desired_domain_width| is greater than |available_domain_width|: the + // minimal username elision allowed by the specifications will not fit; thus + // |desired_domain_width| must be <= |available_domain_width| at all cost. + const int desired_domain_width = + std::min(available_domain_width, + std::max(available_pixel_width - full_username_width, + available_pixel_width / 2)); + domain = ElideText(domain, font_list, desired_domain_width, + ELIDE_IN_MIDDLE); + // Failing to elide the domain such that at least one character remains + // (other than the ellipsis itself) remains: return a single ellipsis. + if (domain.length() <= 1U) + return string16(kEllipsisUTF16); + } + + // Fit the username in the remaining width (at this point the elided username + // is guaranteed to fit with at least one character remaining given all the + // precautions taken earlier). + available_pixel_width -= gfx::GetStringWidth(domain, font_list); + username = ElideText(username, font_list, available_pixel_width, + ELIDE_AT_END); + + return username + kAtSignUTF16 + domain; +} + +string16 ElideEmail(const string16& email, + const gfx::Font& font, + int available_pixel_width) { + return ElideEmail(email, gfx::FontList(font), available_pixel_width); +} + +// TODO(pkasting): http://crbug.com/77883 This whole function gets +// kerning/ligatures/etc. issues potentially wrong by assuming that the width of +// a rendered string is always the sum of the widths of its substrings. Also I +// suspect it could be made simpler. +string16 ElideUrl(const GURL& url, + const gfx::FontList& font_list, + int available_pixel_width, + const std::string& languages) { + // Get a formatted string and corresponding parsing of the url. + url_parse::Parsed parsed; + const string16 url_string = + net::FormatUrl(url, languages, net::kFormatUrlOmitAll, + net::UnescapeRule::SPACES, &parsed, NULL, NULL); + if (available_pixel_width <= 0) + return url_string; + + // If non-standard, return plain eliding. + if (!url.IsStandard()) + return ElideText(url_string, font_list, available_pixel_width, + ELIDE_AT_END); + + // Now start eliding url_string to fit within available pixel width. + // Fist pass - check to see whether entire url_string fits. + const int pixel_width_url_string = gfx::GetStringWidth(url_string, font_list); + if (available_pixel_width >= pixel_width_url_string) + return url_string; + + // Get the path substring, including query and reference. + const size_t path_start_index = parsed.path.begin; + const size_t path_len = parsed.path.len; + string16 url_path_query_etc = url_string.substr(path_start_index); + string16 url_path = url_string.substr(path_start_index, path_len); + + // Return general elided text if url minus the query fits. + const string16 url_minus_query = + url_string.substr(0, path_start_index + path_len); + if (available_pixel_width >= gfx::GetStringWidth(url_minus_query, font_list)) + return ElideText(url_string, font_list, available_pixel_width, + ELIDE_AT_END); + + // Get Host. + string16 url_host = UTF8ToUTF16(url.host()); + + // Get domain and registry information from the URL. + string16 url_domain = UTF8ToUTF16( + net::registry_controlled_domains::GetDomainAndRegistry( + url, net::registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES)); + if (url_domain.empty()) + url_domain = url_host; + + // Add port if required. + if (!url.port().empty()) { + url_host += UTF8ToUTF16(":" + url.port()); + url_domain += UTF8ToUTF16(":" + url.port()); + } + + // Get sub domain. + string16 url_subdomain; + const size_t domain_start_index = url_host.find(url_domain); + if (domain_start_index != string16::npos) + url_subdomain = url_host.substr(0, domain_start_index); + const string16 kWwwPrefix = UTF8ToUTF16("www."); + if ((url_subdomain == kWwwPrefix || url_subdomain.empty() || + url.SchemeIsFile())) { + url_subdomain.clear(); + } + + // If this is a file type, the path is now defined as everything after ":". + // For example, "C:/aa/aa/bb", the path is "/aa/bb/cc". Interesting, the + // domain is now C: - this is a nice hack for eliding to work pleasantly. + if (url.SchemeIsFile()) { + // Split the path string using ":" + std::vector<string16> file_path_split; + base::SplitString(url_path, ':', &file_path_split); + if (file_path_split.size() > 1) { // File is of type "file:///C:/.." + url_host.clear(); + url_domain.clear(); + url_subdomain.clear(); + + const string16 kColon = UTF8ToUTF16(":"); + url_host = url_domain = file_path_split.at(0).substr(1) + kColon; + url_path_query_etc = url_path = file_path_split.at(1); + } + } + + // Second Pass - remove scheme - the rest fits. + const int pixel_width_url_host = gfx::GetStringWidth(url_host, font_list); + const int pixel_width_url_path = gfx::GetStringWidth(url_path_query_etc, + font_list); + if (available_pixel_width >= + pixel_width_url_host + pixel_width_url_path) + return url_host + url_path_query_etc; + + // Third Pass: Subdomain, domain and entire path fits. + const int pixel_width_url_domain = gfx::GetStringWidth(url_domain, font_list); + const int pixel_width_url_subdomain = gfx::GetStringWidth(url_subdomain, + font_list); + if (available_pixel_width >= + pixel_width_url_subdomain + pixel_width_url_domain + + pixel_width_url_path) + return url_subdomain + url_domain + url_path_query_etc; + + // Query element. + string16 url_query; + const int kPixelWidthDotsTrailer = gfx::GetStringWidth( + string16(kEllipsisUTF16), font_list); + if (parsed.query.is_nonempty()) { + url_query = UTF8ToUTF16("?") + url_string.substr(parsed.query.begin); + if (available_pixel_width >= + (pixel_width_url_subdomain + pixel_width_url_domain + + pixel_width_url_path - gfx::GetStringWidth(url_query, font_list))) { + return ElideText(url_subdomain + url_domain + url_path_query_etc, + font_list, available_pixel_width, ELIDE_AT_END); + } + } + + // Parse url_path using '/'. + std::vector<string16> url_path_elements; + base::SplitString(url_path, kForwardSlash, &url_path_elements); + + // Get filename - note that for a path ending with / + // such as www.google.com/intl/ads/, the file name is ads/. + size_t url_path_number_of_elements = url_path_elements.size(); + DCHECK(url_path_number_of_elements != 0); + string16 url_filename; + if ((url_path_elements.at(url_path_number_of_elements - 1)).length() > 0) { + url_filename = *(url_path_elements.end() - 1); + } else if (url_path_number_of_elements > 1) { // Path ends with a '/'. + url_filename = url_path_elements.at(url_path_number_of_elements - 2) + + kForwardSlash; + url_path_number_of_elements--; + } + DCHECK(url_path_number_of_elements != 0); + + const size_t kMaxNumberOfUrlPathElementsAllowed = 1024; + if (url_path_number_of_elements <= 1 || + url_path_number_of_elements > kMaxNumberOfUrlPathElementsAllowed) { + // No path to elide, or too long of a path (could overflow in loop below) + // Just elide this as a text string. + return ElideText(url_subdomain + url_domain + url_path_query_etc, font_list, + available_pixel_width, ELIDE_AT_END); + } + + // Start eliding the path and replacing elements by ".../". + const string16 kEllipsisAndSlash = string16(kEllipsisUTF16) + kForwardSlash; + const int pixel_width_ellipsis_slash = gfx::GetStringWidth(kEllipsisAndSlash, + font_list); + + // Check with both subdomain and domain. + string16 elided_path = + ElideComponentizedPath(url_subdomain + url_domain, url_path_elements, + url_filename, url_query, font_list, + available_pixel_width); + if (!elided_path.empty()) + return elided_path; + + // Check with only domain. + // If a subdomain is present, add an ellipsis before domain. + // This is added only if the subdomain pixel width is larger than + // the pixel width of kEllipsis. Otherwise, subdomain remains, + // which means that this case has been resolved earlier. + string16 url_elided_domain = url_subdomain + url_domain; + if (pixel_width_url_subdomain > kPixelWidthDotsTrailer) { + if (!url_subdomain.empty()) + url_elided_domain = kEllipsisAndSlash[0] + url_domain; + else + url_elided_domain = url_domain; + + elided_path = ElideComponentizedPath(url_elided_domain, url_path_elements, + url_filename, url_query, font_list, + available_pixel_width); + + if (!elided_path.empty()) + return elided_path; + } + + // Return elided domain/.../filename anyway. + string16 final_elided_url_string(url_elided_domain); + const int url_elided_domain_width = gfx::GetStringWidth(url_elided_domain, + font_list); + + // A hack to prevent trailing ".../...". + if ((available_pixel_width - url_elided_domain_width) > + pixel_width_ellipsis_slash + kPixelWidthDotsTrailer + + gfx::GetStringWidth(ASCIIToUTF16("UV"), font_list)) { + final_elided_url_string += BuildPathFromComponents(string16(), + url_path_elements, url_filename, 1); + } else { + final_elided_url_string += url_path; + } + + return ElideText(final_elided_url_string, font_list, available_pixel_width, + ELIDE_AT_END); +} + +string16 ElideUrl(const GURL& url, + const gfx::Font& font, + int available_pixel_width, + const std::string& languages) { + return ElideUrl(url, gfx::FontList(font), available_pixel_width, languages); +} + +string16 ElideFilename(const base::FilePath& filename, + const gfx::FontList& font_list, + int available_pixel_width) { +#if defined(OS_WIN) + string16 filename_utf16 = filename.value(); + string16 extension = filename.Extension(); + string16 rootname = filename.BaseName().RemoveExtension().value(); +#elif defined(OS_POSIX) + string16 filename_utf16 = WideToUTF16(base::SysNativeMBToWide( + filename.value())); + string16 extension = WideToUTF16(base::SysNativeMBToWide( + filename.Extension())); + string16 rootname = WideToUTF16(base::SysNativeMBToWide( + filename.BaseName().RemoveExtension().value())); +#endif + + const int full_width = gfx::GetStringWidth(filename_utf16, font_list); + if (full_width <= available_pixel_width) + return base::i18n::GetDisplayStringInLTRDirectionality(filename_utf16); + + if (rootname.empty() || extension.empty()) { + const string16 elided_name = ElideText(filename_utf16, font_list, + available_pixel_width, ELIDE_AT_END); + return base::i18n::GetDisplayStringInLTRDirectionality(elided_name); + } + + const int ext_width = gfx::GetStringWidth(extension, font_list); + const int root_width = gfx::GetStringWidth(rootname, font_list); + + // We may have trimmed the path. + if (root_width + ext_width <= available_pixel_width) { + const string16 elided_name = rootname + extension; + return base::i18n::GetDisplayStringInLTRDirectionality(elided_name); + } + + if (ext_width >= available_pixel_width) { + const string16 elided_name = ElideText(rootname + extension, font_list, + available_pixel_width, + ELIDE_IN_MIDDLE); + return base::i18n::GetDisplayStringInLTRDirectionality(elided_name); + } + + int available_root_width = available_pixel_width - ext_width; + string16 elided_name = + ElideText(rootname, font_list, available_root_width, ELIDE_AT_END); + elided_name += extension; + return base::i18n::GetDisplayStringInLTRDirectionality(elided_name); +} + +string16 ElideFilename(const base::FilePath& filename, + const gfx::Font& font, + int available_pixel_width) { + return ElideFilename(filename, gfx::FontList(font), available_pixel_width); +} + +string16 ElideText(const string16& text, + const gfx::FontList& font_list, + int available_pixel_width, + ElideBehavior elide_behavior) { + if (text.empty()) + return text; + + const int current_text_pixel_width = gfx::GetStringWidth(text, font_list); + const bool elide_in_middle = (elide_behavior == ELIDE_IN_MIDDLE); + const bool insert_ellipsis = (elide_behavior != TRUNCATE_AT_END); + + const string16 ellipsis = string16(kEllipsisUTF16); + StringSlicer slicer(text, ellipsis, elide_in_middle); + + // Pango will return 0 width for absurdly long strings. Cut the string in + // half and try again. + // This is caused by an int overflow in Pango (specifically, in + // pango_glyph_string_extents_range). It's actually more subtle than just + // returning 0, since on super absurdly long strings, the int can wrap and + // return positive numbers again. Detecting that is probably not worth it + // (eliding way too much from a ridiculous string is probably still + // ridiculous), but we should check other widths for bogus values as well. + if (current_text_pixel_width <= 0 && !text.empty()) { + const string16 cut = slicer.CutString(text.length() / 2, false); + return ElideText(cut, font_list, available_pixel_width, elide_behavior); + } + + if (current_text_pixel_width <= available_pixel_width) + return text; + + if (insert_ellipsis && + gfx::GetStringWidth(ellipsis, font_list) > available_pixel_width) + return string16(); + + // Use binary search to compute the elided text. + size_t lo = 0; + size_t hi = text.length() - 1; + size_t guess; + for (guess = (lo + hi) / 2; lo <= hi; guess = (lo + hi) / 2) { + // We check the length of the whole desired string at once to ensure we + // handle kerning/ligatures/etc. correctly. + const string16 cut = slicer.CutString(guess, insert_ellipsis); + const int guess_length = gfx::GetStringWidth(cut, font_list); + // Check again that we didn't hit a Pango width overflow. If so, cut the + // current string in half and start over. + if (guess_length <= 0) { + return ElideText(slicer.CutString(guess / 2, false), + font_list, available_pixel_width, elide_behavior); + } + if (guess_length > available_pixel_width) + hi = guess - 1; + else + lo = guess + 1; + } + + return slicer.CutString(guess, insert_ellipsis); +} + +string16 ElideText(const string16& text, + const gfx::Font& font, + int available_pixel_width, + ElideBehavior elide_behavior) { + return ElideText(text, gfx::FontList(font), available_pixel_width, + elide_behavior); +} + +SortedDisplayURL::SortedDisplayURL(const GURL& url, + const std::string& languages) { + net::AppendFormattedHost(url, languages, &sort_host_); + string16 host_minus_www = net::StripWWW(sort_host_); + url_parse::Parsed parsed; + display_url_ = + net::FormatUrl(url, languages, net::kFormatUrlOmitAll, + net::UnescapeRule::SPACES, &parsed, &prefix_end_, NULL); + if (sort_host_.length() > host_minus_www.length()) { + prefix_end_ += sort_host_.length() - host_minus_www.length(); + sort_host_.swap(host_minus_www); + } +} + +SortedDisplayURL::SortedDisplayURL() : prefix_end_(0) { +} + +SortedDisplayURL::~SortedDisplayURL() { +} + +int SortedDisplayURL::Compare(const SortedDisplayURL& other, + icu::Collator* collator) const { + // Compare on hosts first. The host won't contain 'www.'. + UErrorCode compare_status = U_ZERO_ERROR; + UCollationResult host_compare_result = collator->compare( + static_cast<const UChar*>(sort_host_.c_str()), + static_cast<int>(sort_host_.length()), + static_cast<const UChar*>(other.sort_host_.c_str()), + static_cast<int>(other.sort_host_.length()), + compare_status); + DCHECK(U_SUCCESS(compare_status)); + if (host_compare_result != 0) + return host_compare_result; + + // Hosts match, compare on the portion of the url after the host. + string16 path = this->AfterHost(); + string16 o_path = other.AfterHost(); + compare_status = U_ZERO_ERROR; + UCollationResult path_compare_result = collator->compare( + static_cast<const UChar*>(path.c_str()), + static_cast<int>(path.length()), + static_cast<const UChar*>(o_path.c_str()), + static_cast<int>(o_path.length()), + compare_status); + DCHECK(U_SUCCESS(compare_status)); + if (path_compare_result != 0) + return path_compare_result; + + // Hosts and paths match, compare on the complete url. This'll push the www. + // ones to the end. + compare_status = U_ZERO_ERROR; + UCollationResult display_url_compare_result = collator->compare( + static_cast<const UChar*>(display_url_.c_str()), + static_cast<int>(display_url_.length()), + static_cast<const UChar*>(other.display_url_.c_str()), + static_cast<int>(other.display_url_.length()), + compare_status); + DCHECK(U_SUCCESS(compare_status)); + return display_url_compare_result; +} + +string16 SortedDisplayURL::AfterHost() const { + const size_t slash_index = display_url_.find(sort_host_, prefix_end_); + if (slash_index == string16::npos) { + NOTREACHED(); + return string16(); + } + return display_url_.substr(slash_index + sort_host_.length()); +} + +bool ElideString(const string16& input, int max_len, string16* output) { + DCHECK_GE(max_len, 0); + if (static_cast<int>(input.length()) <= max_len) { + output->assign(input); + return false; + } + + switch (max_len) { + case 0: + output->clear(); + break; + case 1: + output->assign(input.substr(0, 1)); + break; + case 2: + output->assign(input.substr(0, 2)); + break; + case 3: + output->assign(input.substr(0, 1) + ASCIIToUTF16(".") + + input.substr(input.length() - 1)); + break; + case 4: + output->assign(input.substr(0, 1) + ASCIIToUTF16("..") + + input.substr(input.length() - 1)); + break; + default: { + int rstr_len = (max_len - 3) / 2; + int lstr_len = rstr_len + ((max_len - 3) % 2); + output->assign(input.substr(0, lstr_len) + ASCIIToUTF16("...") + + input.substr(input.length() - rstr_len)); + break; + } + } + + return true; +} + +namespace { + +// Internal class used to track progress of a rectangular string elide +// operation. Exists so the top-level ElideRectangleString() function +// can be broken into smaller methods sharing this state. +class RectangleString { + public: + RectangleString(size_t max_rows, size_t max_cols, + bool strict, string16 *output) + : max_rows_(max_rows), + max_cols_(max_cols), + current_row_(0), + current_col_(0), + strict_(strict), + suppressed_(false), + output_(output) {} + + // Perform deferred initializations following creation. Must be called + // before any input can be added via AddString(). + void Init() { output_->clear(); } + + // Add an input string, reformatting to fit the desired dimensions. + // AddString() may be called multiple times to concatenate together + // multiple strings into the region (the current caller doesn't do + // this, however). + void AddString(const string16& input); + + // Perform any deferred output processing. Must be called after the + // last AddString() call has occurred. + bool Finalize(); + + private: + // Add a line to the rectangular region at the current position, + // either by itself or by breaking it into words. + void AddLine(const string16& line); + + // Add a word to the rectangular region at the current position, + // either by itself or by breaking it into characters. + void AddWord(const string16& word); + + // Add text to the output string if the rectangular boundaries + // have not been exceeded, advancing the current position. + void Append(const string16& string); + + // Set the current position to the beginning of the next line. If + // |output| is true, add a newline to the output string if the rectangular + // boundaries have not been exceeded. If |output| is false, we assume + // some other mechanism will (likely) do similar breaking after the fact. + void NewLine(bool output); + + // Maximum number of rows allowed in the output string. + size_t max_rows_; + + // Maximum number of characters allowed in the output string. + size_t max_cols_; + + // Current row position, always incremented and may exceed max_rows_ + // when the input can not fit in the region. We stop appending to + // the output string, however, when this condition occurs. In the + // future, we may want to expose this value to allow the caller to + // determine how many rows would actually be required to hold the + // formatted string. + size_t current_row_; + + // Current character position, should never exceed max_cols_. + size_t current_col_; + + // True when we do whitespace to newline conversions ourselves. + bool strict_; + + // True when some of the input has been truncated. + bool suppressed_; + + // String onto which the output is accumulated. + string16* output_; + + DISALLOW_COPY_AND_ASSIGN(RectangleString); +}; + +void RectangleString::AddString(const string16& input) { + base::i18n::BreakIterator lines(input, + base::i18n::BreakIterator::BREAK_NEWLINE); + if (lines.Init()) { + while (lines.Advance()) + AddLine(lines.GetString()); + } else { + NOTREACHED() << "BreakIterator (lines) init failed"; + } +} + +bool RectangleString::Finalize() { + if (suppressed_) { + output_->append(ASCIIToUTF16("...")); + return true; + } + return false; +} + +void RectangleString::AddLine(const string16& line) { + if (line.length() < max_cols_) { + Append(line); + } else { + base::i18n::BreakIterator words(line, + base::i18n::BreakIterator::BREAK_SPACE); + if (words.Init()) { + while (words.Advance()) + AddWord(words.GetString()); + } else { + NOTREACHED() << "BreakIterator (words) init failed"; + } + } + // Account for naturally-occuring newlines. + ++current_row_; + current_col_ = 0; +} + +void RectangleString::AddWord(const string16& word) { + if (word.length() < max_cols_) { + // Word can be made to fit, no need to fragment it. + if (current_col_ + word.length() >= max_cols_) + NewLine(strict_); + Append(word); + } else { + // Word is so big that it must be fragmented. + int array_start = 0; + int char_start = 0; + base::i18n::UTF16CharIterator chars(&word); + while (!chars.end()) { + // When boundary is hit, add as much as will fit on this line. + if (current_col_ + (chars.char_pos() - char_start) >= max_cols_) { + Append(word.substr(array_start, chars.array_pos() - array_start)); + NewLine(true); + array_start = chars.array_pos(); + char_start = chars.char_pos(); + } + chars.Advance(); + } + // Add the last remaining fragment, if any. + if (array_start != chars.array_pos()) + Append(word.substr(array_start, chars.array_pos() - array_start)); + } +} + +void RectangleString::Append(const string16& string) { + if (current_row_ < max_rows_) + output_->append(string); + else + suppressed_ = true; + current_col_ += string.length(); +} + +void RectangleString::NewLine(bool output) { + if (current_row_ < max_rows_) { + if (output) + output_->append(ASCIIToUTF16("\n")); + } else { + suppressed_ = true; + } + ++current_row_; + current_col_ = 0; +} + +// Internal class used to track progress of a rectangular text elide +// operation. Exists so the top-level ElideRectangleText() function +// can be broken into smaller methods sharing this state. +class RectangleText { + public: + RectangleText(const gfx::FontList& font_list, + int available_pixel_width, + int available_pixel_height, + WordWrapBehavior wrap_behavior, + std::vector<string16>* lines) + : font_list_(font_list), + line_height_(font_list.GetHeight()), + available_pixel_width_(available_pixel_width), + available_pixel_height_(available_pixel_height), + wrap_behavior_(wrap_behavior), + current_width_(0), + current_height_(0), + last_line_ended_in_lf_(false), + lines_(lines), + insufficient_width_(false), + insufficient_height_(false) {} + + // Perform deferred initializions following creation. Must be called + // before any input can be added via AddString(). + void Init() { lines_->clear(); } + + // Add an input string, reformatting to fit the desired dimensions. + // AddString() may be called multiple times to concatenate together + // multiple strings into the region (the current caller doesn't do + // this, however). + void AddString(const string16& input); + + // Perform any deferred output processing. Must be called after the last + // AddString() call has occured. Returns a combination of + // |ReformattingResultFlags| indicating whether the given width or height was + // insufficient, leading to elision or truncation. + int Finalize(); + + private: + // Add a line to the rectangular region at the current position, + // either by itself or by breaking it into words. + void AddLine(const string16& line); + + // Wrap the specified word across multiple lines. + int WrapWord(const string16& word); + + // Add a long word - wrapping, eliding or truncating per the wrap behavior. + int AddWordOverflow(const string16& word); + + // Add a word to the rectangluar region at the current position. + int AddWord(const string16& word); + + // Append the specified |text| to the current output line, incrementing the + // running width by the specified amount. This is an optimization over + // |AddToCurrentLine()| when |text_width| is already known. + void AddToCurrentLineWithWidth(const string16& text, int text_width); + + // Append the specified |text| to the current output line. + void AddToCurrentLine(const string16& text); + + // Set the current position to the beginning of the next line. + bool NewLine(); + + // The font list used for measuring text width. + const gfx::FontList& font_list_; + + // The height of each line of text. + const int line_height_; + + // The number of pixels of available width in the rectangle. + const int available_pixel_width_; + + // The number of pixels of available height in the rectangle. + const int available_pixel_height_; + + // The wrap behavior for words that are too long to fit on a single line. + const WordWrapBehavior wrap_behavior_; + + // The current running width. + int current_width_; + + // The current running height. + int current_height_; + + // The current line of text. + string16 current_line_; + + // Indicates whether the last line ended with \n. + bool last_line_ended_in_lf_; + + // The output vector of lines. + std::vector<string16>* lines_; + + // Indicates whether a word was so long that it had to be truncated or elided + // to fit the available width. + bool insufficient_width_; + + // Indicates whether there were too many lines for the available height. + bool insufficient_height_; + + DISALLOW_COPY_AND_ASSIGN(RectangleText); +}; + +void RectangleText::AddString(const string16& input) { + base::i18n::BreakIterator lines(input, + base::i18n::BreakIterator::BREAK_NEWLINE); + if (lines.Init()) { + while (!insufficient_height_ && lines.Advance()) { + string16 line = lines.GetString(); + // The BREAK_NEWLINE iterator will keep the trailing newline character, + // except in the case of the last line, which may not have one. Remove + // the newline character, if it exists. + last_line_ended_in_lf_ = !line.empty() && line[line.length() - 1] == '\n'; + if (last_line_ended_in_lf_) + line.resize(line.length() - 1); + AddLine(line); + } + } else { + NOTREACHED() << "BreakIterator (lines) init failed"; + } +} + +int RectangleText::Finalize() { + // Remove trailing whitespace from the last line or remove the last line + // completely, if it's just whitespace. + if (!insufficient_height_ && !lines_->empty()) { + TrimWhitespace(lines_->back(), TRIM_TRAILING, &lines_->back()); + if (lines_->back().empty() && !last_line_ended_in_lf_) + lines_->pop_back(); + } + if (last_line_ended_in_lf_) + lines_->push_back(string16()); + return (insufficient_width_ ? INSUFFICIENT_SPACE_HORIZONTAL : 0) | + (insufficient_height_ ? INSUFFICIENT_SPACE_VERTICAL : 0); +} + +void RectangleText::AddLine(const string16& line) { + const int line_width = gfx::GetStringWidth(line, font_list_); + if (line_width <= available_pixel_width_) { + AddToCurrentLineWithWidth(line, line_width); + } else { + // Iterate over positions that are valid to break the line at. In general, + // these are word boundaries but after any punctuation following the word. + base::i18n::BreakIterator words(line, + base::i18n::BreakIterator::BREAK_LINE); + if (words.Init()) { + while (words.Advance()) { + const bool truncate = !current_line_.empty(); + const string16& word = words.GetString(); + const int lines_added = AddWord(word); + if (lines_added) { + if (truncate) { + // Trim trailing whitespace from the line that was added. + const int line = lines_->size() - lines_added; + TrimWhitespace(lines_->at(line), TRIM_TRAILING, &lines_->at(line)); + } + if (ContainsOnlyWhitespace(word)) { + // Skip the first space if the previous line was carried over. + current_width_ = 0; + current_line_.clear(); + } + } + } + } else { + NOTREACHED() << "BreakIterator (words) init failed"; + } + } + // Account for naturally-occuring newlines. + NewLine(); +} + +int RectangleText::WrapWord(const string16& word) { + // Word is so wide that it must be fragmented. + string16 text = word; + int lines_added = 0; + bool first_fragment = true; + while (!insufficient_height_ && !text.empty()) { + string16 fragment = + ElideText(text, font_list_, available_pixel_width_, + TRUNCATE_AT_END); + // At least one character has to be added at every line, even if the + // available space is too small. + if(fragment.empty()) + fragment = text.substr(0, 1); + if (!first_fragment && NewLine()) + lines_added++; + AddToCurrentLine(fragment); + text = text.substr(fragment.length()); + first_fragment = false; + } + return lines_added; +} + +int RectangleText::AddWordOverflow(const string16& word) { + int lines_added = 0; + + // Unless this is the very first word, put it on a new line. + if (!current_line_.empty()) { + if (!NewLine()) + return 0; + lines_added++; + } + + if (wrap_behavior_ == IGNORE_LONG_WORDS) { + current_line_ = word; + current_width_ = available_pixel_width_; + } else if (wrap_behavior_ == WRAP_LONG_WORDS) { + lines_added += WrapWord(word); + } else { + const ElideBehavior elide_behavior = + (wrap_behavior_ == ELIDE_LONG_WORDS ? ELIDE_AT_END : TRUNCATE_AT_END); + const string16 elided_word = + ElideText(word, font_list_, available_pixel_width_, elide_behavior); + AddToCurrentLine(elided_word); + insufficient_width_ = true; + } + + return lines_added; +} + +int RectangleText::AddWord(const string16& word) { + int lines_added = 0; + string16 trimmed; + TrimWhitespace(word, TRIM_TRAILING, &trimmed); + const int trimmed_width = gfx::GetStringWidth(trimmed, font_list_); + if (trimmed_width <= available_pixel_width_) { + // Word can be made to fit, no need to fragment it. + if ((current_width_ + trimmed_width > available_pixel_width_) && NewLine()) + lines_added++; + // Append the non-trimmed word, in case more words are added after. + AddToCurrentLine(word); + } else { + lines_added = AddWordOverflow(wrap_behavior_ == IGNORE_LONG_WORDS ? + trimmed : word); + } + return lines_added; +} + +void RectangleText::AddToCurrentLine(const string16& text) { + AddToCurrentLineWithWidth(text, gfx::GetStringWidth(text, font_list_)); +} + +void RectangleText::AddToCurrentLineWithWidth(const string16& text, + int text_width) { + if (current_height_ >= available_pixel_height_) { + insufficient_height_ = true; + return; + } + current_line_.append(text); + current_width_ += text_width; +} + +bool RectangleText::NewLine() { + bool line_added = false; + if (current_height_ < available_pixel_height_) { + lines_->push_back(current_line_); + current_line_.clear(); + line_added = true; + } else { + insufficient_height_ = true; + } + current_height_ += line_height_; + current_width_ = 0; + return line_added; +} + +} // namespace + +bool ElideRectangleString(const string16& input, size_t max_rows, + size_t max_cols, bool strict, string16* output) { + RectangleString rect(max_rows, max_cols, strict, output); + rect.Init(); + rect.AddString(input); + return rect.Finalize(); +} + +int ElideRectangleText(const string16& input, + const gfx::FontList& font_list, + int available_pixel_width, + int available_pixel_height, + WordWrapBehavior wrap_behavior, + std::vector<string16>* lines) { + RectangleText rect(font_list, + available_pixel_width, + available_pixel_height, + wrap_behavior, + lines); + rect.Init(); + rect.AddString(input); + return rect.Finalize(); +} + +int ElideRectangleText(const string16& input, + const gfx::Font& font, + int available_pixel_width, + int available_pixel_height, + WordWrapBehavior wrap_behavior, + std::vector<string16>* lines) { + return ElideRectangleText(input, gfx::FontList(font), + available_pixel_width, available_pixel_height, + wrap_behavior, lines); +} + +string16 TruncateString(const string16& string, size_t length) { + if (string.size() <= length) + // String fits, return it. + return string; + + if (length == 0) + // No room for the elide string, return an empty string. + return string16(); + + size_t max = length - 1; + + // Added to the end of strings that are too big. + static const char16 kElideString[] = { 0x2026, 0 }; + + if (max == 0) + // Just enough room for the elide string. + return kElideString; + + // Use a line iterator to find the first boundary. + UErrorCode status = U_ZERO_ERROR; + scoped_ptr<icu::RuleBasedBreakIterator> bi( + static_cast<icu::RuleBasedBreakIterator*>( + icu::RuleBasedBreakIterator::createLineInstance( + icu::Locale::getDefault(), status))); + if (U_FAILURE(status)) + return string.substr(0, max) + kElideString; + bi->setText(string.c_str()); + int32_t index = bi->preceding(static_cast<int32_t>(max)); + if (index == icu::BreakIterator::DONE) { + index = static_cast<int32_t>(max); + } else { + // Found a valid break (may be the beginning of the string). Now use + // a character iterator to find the previous non-whitespace character. + icu::StringCharacterIterator char_iterator(string.c_str()); + if (index == 0) { + // No valid line breaks. Start at the end again. This ensures we break + // on a valid character boundary. + index = static_cast<int32_t>(max); + } + char_iterator.setIndex(index); + while (char_iterator.hasPrevious()) { + char_iterator.previous(); + if (!(u_isspace(char_iterator.current()) || + u_charType(char_iterator.current()) == U_CONTROL_CHAR || + u_charType(char_iterator.current()) == U_NON_SPACING_MARK)) { + // Not a whitespace character. Advance the iterator so that we + // include the current character in the truncated string. + char_iterator.next(); + break; + } + } + if (char_iterator.hasPrevious()) { + // Found a valid break point. + index = char_iterator.getIndex(); + } else { + // String has leading whitespace, return the elide string. + return kElideString; + } + } + return string.substr(0, index) + kElideString; +} + +} // namespace gfx diff --git a/ui/gfx/text_elider.h b/ui/gfx/text_elider.h new file mode 100644 index 0000000..008a1cd --- /dev/null +++ b/ui/gfx/text_elider.h @@ -0,0 +1,232 @@ +// Copyright (c) 2012 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. +// +// This file defines utility functions for eliding and formatting UI text. + +#ifndef UI_GFX_TEXT_ELIDER_H_ +#define UI_GFX_TEXT_ELIDER_H_ + +#include <string> +#include <vector> + +#include "base/basictypes.h" +#include "base/strings/string16.h" +#include "third_party/icu/source/common/unicode/uchar.h" +#include "third_party/icu/source/i18n/unicode/coll.h" +#include "ui/gfx/gfx_export.h" + +class GURL; + +namespace base { +class FilePath; +} + +namespace gfx { +class Font; +class FontList; + +UI_EXPORT extern const char kEllipsis[]; +UI_EXPORT extern const char16 kEllipsisUTF16[]; + +// Elides a well-formed email address (e.g. username@domain.com) to fit into +// |available_pixel_width| using the specified |font_list|. +// This function guarantees that the string returned will contain at least one +// character, other than the ellipses, on either side of the '@'. If it is +// impossible to achieve these requirements: only an ellipsis will be returned. +// If possible: this elides only the username portion of the |email|. Otherwise, +// the domain is elided in the middle so that it splits the available width +// equally with the elided username (should the username be short enough that it +// doesn't need half the available width: the elided domain will occupy that +// extra width). +UI_EXPORT string16 ElideEmail(const string16& email, + const gfx::FontList& font_list, + int available_pixel_width); +// Obsolete version. Use the above version which takes gfx::FontList. +UI_EXPORT string16 ElideEmail(const string16& email, + const gfx::Font& font, + int available_pixel_width); + +// This function takes a GURL object and elides it. It returns a string +// which composed of parts from subdomain, domain, path, filename and query. +// A "..." is added automatically at the end if the elided string is bigger +// than the |available_pixel_width|. For |available_pixel_width| == 0, a +// formatted, but un-elided, string is returned. |languages| is a comma +// separated list of ISO 639 language codes and is used to determine what +// characters are understood by a user. It should come from +// |prefs::kAcceptLanguages|. +// +// Note: in RTL locales, if the URL returned by this function is going to be +// displayed in the UI, then it is likely that the string needs to be marked +// as an LTR string (using base::i18n::WrapStringWithLTRFormatting()) so that it +// is displayed properly in an RTL context. Please refer to +// http://crbug.com/6487 for more information. +UI_EXPORT string16 ElideUrl(const GURL& url, + const gfx::FontList& font_list, + int available_pixel_width, + const std::string& languages); +// Obsolete version. Use the above version which takes gfx::FontList. +UI_EXPORT string16 ElideUrl(const GURL& url, + const gfx::Font& font, + int available_pixel_width, + const std::string& languages); + +enum ElideBehavior { + // Add ellipsis at the end of the string. + ELIDE_AT_END, + // Add ellipsis in the middle of the string. + ELIDE_IN_MIDDLE, + // Truncate the end of the string. + TRUNCATE_AT_END +}; + +// Elides |text| to fit in |available_pixel_width| according to the specified +// |elide_behavior|. +UI_EXPORT string16 ElideText(const string16& text, + const gfx::FontList& font_list, + int available_pixel_width, + ElideBehavior elide_behavior); +// Obsolete version. Use the above version which takes gfx::FontList. +UI_EXPORT string16 ElideText(const string16& text, + const gfx::Font& font, + int available_pixel_width, + ElideBehavior elide_behavior); + +// Elide a filename to fit a given pixel width, with an emphasis on not hiding +// the extension unless we have to. If filename contains a path, the path will +// be removed if filename doesn't fit into available_pixel_width. The elided +// filename is forced to have LTR directionality, which means that in RTL UI +// the elided filename is wrapped with LRE (Left-To-Right Embedding) mark and +// PDF (Pop Directional Formatting) mark. +UI_EXPORT string16 ElideFilename(const base::FilePath& filename, + const gfx::FontList& font_list, + int available_pixel_width); +// Obsolete version. Use the above version which takes gfx::FontList. +UI_EXPORT string16 ElideFilename(const base::FilePath& filename, + const gfx::Font& font, + int available_pixel_width); + +// SortedDisplayURL maintains a string from a URL suitable for display to the +// use. SortedDisplayURL also provides a function used for comparing two +// SortedDisplayURLs for use in visually ordering the SortedDisplayURLs. +// +// SortedDisplayURL is relatively cheap and supports value semantics. +class UI_EXPORT SortedDisplayURL { + public: + SortedDisplayURL(const GURL& url, const std::string& languages); + SortedDisplayURL(); + ~SortedDisplayURL(); + + // Compares this SortedDisplayURL to |url| using |collator|. Returns a value + // < 0, = 1 or > 0 as to whether this url is less then, equal to or greater + // than the supplied url. + int Compare(const SortedDisplayURL& other, icu::Collator* collator) const; + + // Returns the display string for the URL. + const string16& display_url() const { return display_url_; } + + private: + // Returns everything after the host. This is used by Compare if the hosts + // match. + string16 AfterHost() const; + + // Host name minus 'www.'. Used by Compare. + string16 sort_host_; + + // End of the prefix (spec and separator) in display_url_. + size_t prefix_end_; + + string16 display_url_; + + DISALLOW_COPY_AND_ASSIGN(SortedDisplayURL); +}; + +// Functions to elide strings when the font information is unknown. As +// opposed to the above functions, the ElideString() and +// ElideRectangleString() functions operate in terms of character units, +// not pixels. + +// If the size of |input| is more than |max_len|, this function returns +// true and |input| is shortened into |output| by removing chars in the +// middle (they are replaced with up to 3 dots, as size permits). +// Ex: ElideString(ASCIIToUTF16("Hello"), 10, &str) puts Hello in str and +// returns false. ElideString(ASCIIToUTF16("Hello my name is Tom"), 10, &str) +// puts "Hell...Tom" in str and returns true. +// TODO(tsepez): Doesn't handle UTF-16 surrogate pairs properly. +// TODO(tsepez): Doesn't handle bidi properly. +UI_EXPORT bool ElideString(const string16& input, int max_len, + string16* output); + +// Reformat |input| into |output| so that it fits into a |max_rows| by +// |max_cols| rectangle of characters. Input newlines are respected, but +// lines that are too long are broken into pieces. If |strict| is true, +// we break first at naturally occuring whitespace boundaries, otherwise +// we assume some other mechanism will do this in approximately the same +// spot after the fact. If the word itself is too long, we always break +// intra-word (respecting UTF-16 surrogate pairs) as necssary. Truncation +// (indicated by an added 3 dots) occurs if the result is still too long. +// Returns true if the input had to be truncated (and not just reformatted). +UI_EXPORT bool ElideRectangleString(const string16& input, size_t max_rows, + size_t max_cols, bool strict, + string16* output); + +// Specifies the word wrapping behavior of |ElideRectangleText()| when a word +// would exceed the available width. +enum WordWrapBehavior { + // Words that are too wide will be put on a new line, but will not be + // truncated or elided. + IGNORE_LONG_WORDS, + + // Words that are too wide will be put on a new line and will be truncated to + // the available width. + TRUNCATE_LONG_WORDS, + + // Words that are too wide will be put on a new line and will be elided to the + // available width. + ELIDE_LONG_WORDS, + + // Words that are too wide will be put on a new line and will be wrapped over + // multiple lines. + WRAP_LONG_WORDS, +}; + +// Indicates whether the |available_pixel_width| by |available_pixel_height| +// rectangle passed to |ElideRectangleText()| had insufficient space to +// accommodate the given |text|, leading to elision or truncation. +enum ReformattingResultFlags { + INSUFFICIENT_SPACE_HORIZONTAL = 1 << 0, + INSUFFICIENT_SPACE_VERTICAL = 1 << 1, +}; + +// Reformats |text| into output vector |lines| so that the resulting text fits +// into an |available_pixel_width| by |available_pixel_height| rectangle with +// the specified |font_list|. Input newlines are respected, but lines that are +// too long are broken into pieces. For words that are too wide to fit on a +// single line, the wrapping behavior can be specified with the |wrap_behavior| +// param. Returns a combination of |ReformattingResultFlags| that indicate +// whether the given rectangle had insufficient space to accommodate |texŧ|, +// leading to elision or truncation (and not just reformatting). +UI_EXPORT int ElideRectangleText(const string16& text, + const gfx::FontList& font_list, + int available_pixel_width, + int available_pixel_height, + WordWrapBehavior wrap_behavior, + std::vector<string16>* lines); +// Obsolete version. Use the above version which takes gfx::FontList. +UI_EXPORT int ElideRectangleText(const string16& text, + const gfx::Font& font, + int available_pixel_width, + int available_pixel_height, + WordWrapBehavior wrap_behavior, + std::vector<string16>* lines); + +// Truncates the string to length characters. This breaks the string at +// the first word break before length, adding the horizontal ellipsis +// character (unicode character 0x2026) to render ... +// The supplied string is returned if the string has length characters or +// less. +UI_EXPORT string16 TruncateString(const string16& string, size_t length); + +} // namespace gfx + +#endif // UI_GFX_TEXT_ELIDER_H_ diff --git a/ui/gfx/text_elider_unittest.cc b/ui/gfx/text_elider_unittest.cc new file mode 100644 index 0000000..a9e0fb5 --- /dev/null +++ b/ui/gfx/text_elider_unittest.cc @@ -0,0 +1,934 @@ +// Copyright (c) 2012 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. +// +// Unit tests for eliding and formatting utility functions. + +#include "ui/gfx/text_elider.h" + +#include "base/files/file_path.h" +#include "base/i18n/rtl.h" +#include "base/memory/scoped_ptr.h" +#include "base/strings/string_util.h" +#include "base/strings/utf_string_conversions.h" +#include "testing/gtest/include/gtest/gtest.h" +#include "ui/gfx/font.h" +#include "url/gurl.h" + +namespace gfx { + +namespace { + +struct Testcase { + const std::string input; + const std::string output; +}; + +struct FileTestcase { + const base::FilePath::StringType input; + const std::string output; +}; + +struct UTF16Testcase { + const string16 input; + const string16 output; +}; + +struct TestData { + const std::string a; + const std::string b; + const int compare_result; +}; + +void RunUrlTest(Testcase* testcases, size_t num_testcases) { + static const gfx::Font 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(UTF8ToUTF16(testcases[i].output), + ElideUrl(url, font, + font.GetStringWidth(UTF8ToUTF16(testcases[i].output)), + std::string())); + } +} + +} // namespace + +// TODO(ios): Complex eliding is off by one for some of those tests on iOS. +// See crbug.com/154019 +#if defined(OS_IOS) +#define MAYBE_ElideEmail DISABLED_ElideEmail +#else +#define MAYBE_ElideEmail ElideEmail +#endif +TEST(TextEliderTest, MAYBE_ElideEmail) { + const std::string kEllipsisStr(kEllipsis); + + // Test emails and their expected elided forms (from which the available + // widths will be derived). + // For elided forms in which both the username and domain must be elided: + // the result (how many characters are left on each side) can be font + // dependent. To avoid this, the username is prefixed with the characters + // expected to remain in the domain. + Testcase testcases[] = { + {"g@g.c", "g@g.c"}, + {"g@g.c", kEllipsisStr}, + {"ga@co.ca", "ga@c" + kEllipsisStr + "a"}, + {"short@small.com", "s" + kEllipsisStr + "@s" + kEllipsisStr}, + {"short@small.com", "s" + kEllipsisStr + "@small.com"}, + {"short@longbutlotsofspace.com", "short@longbutlotsofspace.com"}, + {"short@longbutnotverymuchspace.com", + "short@long" + kEllipsisStr + ".com"}, + {"la_short@longbutverytightspace.ca", + "la" + kEllipsisStr + "@l" + kEllipsisStr + "a"}, + {"longusername@gmail.com", "long" + kEllipsisStr + "@gmail.com"}, + {"elidetothemax@justfits.com", "e" + kEllipsisStr + "@justfits.com"}, + {"thatom_somelongemail@thatdoesntfit.com", + "thatom" + kEllipsisStr + "@tha" + kEllipsisStr + "om"}, + {"namefits@butthedomaindoesnt.com", + "namefits@butthedo" + kEllipsisStr + "snt.com"}, + {"widthtootight@nospace.com", kEllipsisStr}, + {"nospaceforusername@l", kEllipsisStr}, + {"little@littlespace.com", "l" + kEllipsisStr + "@l" + kEllipsisStr}, + {"l@llllllllllllllllllllllll.com", "l@lllll" + kEllipsisStr + ".com"}, + {"messed\"up@whyanat\"++@notgoogley.com", + "messed\"up@whyanat\"++@notgoogley.com"}, + {"messed\"up@whyanat\"++@notgoogley.com", + "messed\"up@why" + kEllipsisStr + "@notgoogley.com"}, + {"noca_messed\"up@whyanat\"++@notgoogley.ca", + "noca" + kEllipsisStr + "@no" + kEllipsisStr + "ca"}, + {"at\"@@@@@@@@@...@@.@.@.@@@\"@madness.com", + "at\"@@@@@@@@@...@@.@." + kEllipsisStr + "@madness.com"}, + // Special case: "m..." takes more than half of the available width; thus + // the domain must elide to "l..." and not "l...l" as it must allow enough + // space for the minimal username elision although its half of the + // available width would normally allow it to elide to "l...l". + {"mmmmm@llllllllll", "m" + kEllipsisStr + "@l" + kEllipsisStr}, + }; + + const gfx::Font font; + for (size_t i = 0; i < arraysize(testcases); ++i) { + const string16 expected_output = UTF8ToUTF16(testcases[i].output); + EXPECT_EQ(expected_output, + ElideEmail(UTF8ToUTF16(testcases[i].input), + font, + font.GetStringWidth(expected_output))); + } +} + +TEST(TextEliderTest, ElideEmailMoreSpace) { + const int test_width_factors[] = { + 100, + 10000, + 1000000, + }; + const std::string test_emails[] = { + "a@c", + "test@email.com", + "short@verysuperdupperlongdomain.com", + "supermegalongusername@withasuperlonnnggggdomain.gouv.qc.ca", + }; + + const gfx::Font font; + for (size_t i = 0; i < arraysize(test_width_factors); ++i) { + const int test_width = test_width_factors[i] * + font.GetAverageCharacterWidth(); + for (size_t j = 0; j < arraysize(test_emails); ++j) { + // Extra space is available: the email should not be elided. + const string16 test_email = UTF8ToUTF16(test_emails[j]); + EXPECT_EQ(test_email, ElideEmail(test_email, font, test_width)); + } + } +} + +// Test eliding of commonplace URLs. +TEST(TextEliderTest, TestGeneralEliding) { + const std::string kEllipsisStr(kEllipsis); + Testcase testcases[] = { + {"http://www.google.com/intl/en/ads/", + "www.google.com/intl/en/ads/"}, + {"http://www.google.com/intl/en/ads/", "www.google.com/intl/en/ads/"}, + {"http://www.google.com/intl/en/ads/", + "google.com/intl/" + kEllipsisStr + "/ads/"}, + {"http://www.google.com/intl/en/ads/", + "google.com/" + kEllipsisStr + "/ads/"}, + {"http://www.google.com/intl/en/ads/", "google.com/" + kEllipsisStr}, + {"http://www.google.com/intl/en/ads/", "goog" + kEllipsisStr}, + {"https://subdomain.foo.com/bar/filename.html", + "subdomain.foo.com/bar/filename.html"}, + {"https://subdomain.foo.com/bar/filename.html", + "subdomain.foo.com/" + kEllipsisStr + "/filename.html"}, + {"http://subdomain.foo.com/bar/filename.html", + kEllipsisStr + "foo.com/" + kEllipsisStr + "/filename.html"}, + {"http://www.google.com/intl/en/ads/?aLongQueryWhichIsNotRequired", + "www.google.com/intl/en/ads/?aLongQ" + kEllipsisStr}, + }; + + RunUrlTest(testcases, arraysize(testcases)); +} + +// When there is very little space available, the elision code will shorten +// both path AND file name to an ellipsis - ".../...". To avoid this result, +// there is a hack in place that simply treats them as one string in this +// case. +TEST(TextEliderTest, TestTrailingEllipsisSlashEllipsisHack) { + const std::string kEllipsisStr(kEllipsis); + + // Very little space, would cause double ellipsis. + gfx::Font font; + GURL url("http://battersbox.com/directory/foo/peter_paul_and_mary.html"); + int available_width = font.GetStringWidth( + UTF8ToUTF16("battersbox.com/" + kEllipsisStr + "/" + kEllipsisStr)); + + // Create the expected string, after elision. Depending on font size, the + // directory might become /dir... or /di... or/d... - it never should be + // shorter than that. (If it is, the font considers d... to be longer + // than .../... - that should never happen). + ASSERT_GT(font.GetStringWidth(UTF8ToUTF16(kEllipsisStr + "/" + kEllipsisStr)), + font.GetStringWidth(UTF8ToUTF16("d" + kEllipsisStr))); + GURL long_url("http://battersbox.com/directorynameisreallylongtoforcetrunc"); + string16 expected = ElideUrl(long_url, font, available_width, std::string()); + // 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, + ElideUrl(url, font, available_width, std::string())); + + // More space available - elide directories, partially elide filename. + Testcase testcases[] = { + {"http://battersbox.com/directory/foo/peter_paul_and_mary.html", + "battersbox.com/" + kEllipsisStr + "/peter" + kEllipsisStr}, + }; + RunUrlTest(testcases, arraysize(testcases)); +} + +// Test eliding of empty strings, URLs with ports, passwords, queries, etc. +TEST(TextEliderTest, TestMoreEliding) { + const std::string kEllipsisStr(kEllipsis); + Testcase testcases[] = { + {"http://www.google.com/foo?bar", "www.google.com/foo?bar"}, + {"http://xyz.google.com/foo?bar", "xyz.google.com/foo?" + kEllipsisStr}, + {"http://xyz.google.com/foo?bar", "xyz.google.com/foo" + kEllipsisStr}, + {"http://xyz.google.com/foo?bar", "xyz.google.com/fo" + kEllipsisStr}, + {"http://a.b.com/pathname/c?d", "a.b.com/" + kEllipsisStr + "/c?d"}, + {"", ""}, + {"http://foo.bar..example.com...hello/test/filename.html", + "foo.bar..example.com...hello/" + kEllipsisStr + "/filename.html"}, + {"http://foo.bar../", "foo.bar.."}, + {"http://xn--1lq90i.cn/foo", "\xe5\x8c\x97\xe4\xba\xac.cn/foo"}, + {"http://me:mypass@secrethost.com:99/foo?bar#baz", + "secrethost.com:99/foo?bar#baz"}, + {"http://me:mypass@ss%xxfdsf.com/foo", "ss%25xxfdsf.com/foo"}, + {"mailto:elgoato@elgoato.com", "mailto:elgoato@elgoato.com"}, + {"javascript:click(0)", "javascript:click(0)"}, + {"https://chess.eecs.berkeley.edu:4430/login/arbitfilename", + "chess.eecs.berkeley.edu:4430/login/arbitfilename"}, + {"https://chess.eecs.berkeley.edu:4430/login/arbitfilename", + kEllipsisStr + "berkeley.edu:4430/" + kEllipsisStr + "/arbitfilename"}, + + // Unescaping. + {"http://www/%E4%BD%A0%E5%A5%BD?q=%E4%BD%A0%E5%A5%BD#\xe4\xbd\xa0", + "www/\xe4\xbd\xa0\xe5\xa5\xbd?q=\xe4\xbd\xa0\xe5\xa5\xbd#\xe4\xbd\xa0"}, + + // 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", + "www/%E4%A0%E5%A5%BD?q=\xe4\xbd\xa0\xe5\xa5\xbd#\xe4\xbd\xa0"}, + }; + + RunUrlTest(testcases, arraysize(testcases)); +} + +// Test eliding of file: URLs. +TEST(TextEliderTest, TestFileURLEliding) { + const std::string kEllipsisStr(kEllipsis); + Testcase testcases[] = { + {"file:///C:/path1/path2/path3/filename", + "file:///C:/path1/path2/path3/filename"}, + {"file:///C:/path1/path2/path3/filename", + "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", + "C:/path1/path2/" + kEllipsisStr + "/filename"}, + {"file:///C:path1/path2/path3/filename", + "C:/path1/" + kEllipsisStr + "/filename"}, + {"file:///C:path1/path2/path3/filename", + "C:/" + kEllipsisStr + "/filename"}, +#endif + {"file://filer/foo/bar/file", "filer/foo/bar/file"}, + {"file://filer/foo/bar/file", "filer/foo/" + kEllipsisStr + "/file"}, + {"file://filer/foo/bar/file", "filer/" + kEllipsisStr + "/file"}, + }; + + RunUrlTest(testcases, arraysize(testcases)); +} + +// TODO(ios): Complex eliding is off by one for some of those tests on iOS. +// See crbug.com/154019 +#if defined(OS_IOS) +#define MAYBE_TestFilenameEliding DISABLED_TestFilenameEliding +#else +#define MAYBE_TestFilenameEliding TestFilenameEliding +#endif +TEST(TextEliderTest, MAYBE_TestFilenameEliding) { + const std::string kEllipsisStr(kEllipsis); + const base::FilePath::StringType kPathSeparator = + base::FilePath::StringType().append(1, base::FilePath::kSeparators[0]); + + FileTestcase testcases[] = { + {FILE_PATH_LITERAL(""), ""}, + {FILE_PATH_LITERAL("."), "."}, + {FILE_PATH_LITERAL("filename.exe"), "filename.exe"}, + {FILE_PATH_LITERAL(".longext"), ".longext"}, + {FILE_PATH_LITERAL("pie"), "pie"}, + {FILE_PATH_LITERAL("c:") + kPathSeparator + FILE_PATH_LITERAL("path") + + kPathSeparator + FILE_PATH_LITERAL("filename.pie"), + "filename.pie"}, + {FILE_PATH_LITERAL("c:") + kPathSeparator + FILE_PATH_LITERAL("path") + + kPathSeparator + FILE_PATH_LITERAL("longfilename.pie"), + "long" + kEllipsisStr + ".pie"}, + {FILE_PATH_LITERAL("http://path.com/filename.pie"), "filename.pie"}, + {FILE_PATH_LITERAL("http://path.com/longfilename.pie"), + "long" + kEllipsisStr + ".pie"}, + {FILE_PATH_LITERAL("piesmashingtacularpants"), "pie" + kEllipsisStr}, + {FILE_PATH_LITERAL(".piesmashingtacularpants"), ".pie" + kEllipsisStr}, + {FILE_PATH_LITERAL("cheese."), "cheese."}, + {FILE_PATH_LITERAL("file name.longext"), + "file" + kEllipsisStr + ".longext"}, + {FILE_PATH_LITERAL("fil ename.longext"), + "fil " + kEllipsisStr + ".longext"}, + {FILE_PATH_LITERAL("filename.longext"), + "file" + kEllipsisStr + ".longext"}, + {FILE_PATH_LITERAL("filename.middleext.longext"), + "filename.mid" + kEllipsisStr + ".longext"}, + {FILE_PATH_LITERAL("filename.superduperextremelylongext"), + "filename.sup" + kEllipsisStr + "emelylongext"}, + {FILE_PATH_LITERAL("filenamereallylongtext.superduperextremelylongext"), + "filenamereall" + kEllipsisStr + "emelylongext"}, + {FILE_PATH_LITERAL("file.name.really.long.text.superduperextremelylongext"), + "file.name.re" + kEllipsisStr + "emelylongext"} + }; + + static const gfx::Font font; + for (size_t i = 0; i < arraysize(testcases); ++i) { + base::FilePath filepath(testcases[i].input); + string16 expected = UTF8ToUTF16(testcases[i].output); + expected = base::i18n::GetDisplayStringInLTRDirectionality(expected); + EXPECT_EQ(expected, ElideFilename(filepath, + font, + font.GetStringWidth(UTF8ToUTF16(testcases[i].output)))); + } +} + +TEST(TextEliderTest, ElideTextTruncate) { + const gfx::Font font; + const int kTestWidth = font.GetStringWidth(ASCIIToUTF16("Test")); + struct TestData { + const char* input; + int width; + const char* output; + } cases[] = { + { "", 0, "" }, + { "Test", 0, "" }, + { "", kTestWidth, "" }, + { "Tes", kTestWidth, "Tes" }, + { "Test", kTestWidth, "Test" }, + { "Tests", kTestWidth, "Test" }, + }; + + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + string16 result = ElideText(UTF8ToUTF16(cases[i].input), font, + cases[i].width, TRUNCATE_AT_END); + EXPECT_EQ(cases[i].output, UTF16ToUTF8(result)); + } +} + +TEST(TextEliderTest, ElideTextEllipsis) { + const gfx::Font font; + const int kTestWidth = font.GetStringWidth(ASCIIToUTF16("Test")); + const char* kEllipsis = "\xE2\x80\xA6"; + const int kEllipsisWidth = font.GetStringWidth(UTF8ToUTF16(kEllipsis)); + struct TestData { + const char* input; + int width; + const char* output; + } cases[] = { + { "", 0, "" }, + { "Test", 0, "" }, + { "Test", kEllipsisWidth, kEllipsis }, + { "", kTestWidth, "" }, + { "Tes", kTestWidth, "Tes" }, + { "Test", kTestWidth, "Test" }, + }; + + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + string16 result = ElideText(UTF8ToUTF16(cases[i].input), font, + cases[i].width, ELIDE_AT_END); + EXPECT_EQ(cases[i].output, UTF16ToUTF8(result)); + } +} + +// Checks that all occurrences of |first_char| are followed by |second_char| and +// all occurrences of |second_char| are preceded by |first_char| in |text|. +static void CheckSurrogatePairs(const string16& text, + char16 first_char, + char16 second_char) { + size_t index = text.find_first_of(first_char); + while (index != string16::npos) { + EXPECT_LT(index, text.length() - 1); + EXPECT_EQ(second_char, text[index + 1]); + index = text.find_first_of(first_char, index + 1); + } + index = text.find_first_of(second_char); + while (index != string16::npos) { + EXPECT_GT(index, 0U); + EXPECT_EQ(first_char, text[index - 1]); + index = text.find_first_of(second_char, index + 1); + } +} + +TEST(TextEliderTest, ElideTextSurrogatePairs) { + const gfx::Font font; + // The below is 'MUSICAL SYMBOL G CLEF', which is represented in UTF-16 as + // two characters forming a surrogate pair 0x0001D11E. + const std::string kSurrogate = "\xF0\x9D\x84\x9E"; + const string16 kTestString = + UTF8ToUTF16(kSurrogate + "ab" + kSurrogate + kSurrogate + "cd"); + const int kTestStringWidth = font.GetStringWidth(kTestString); + const char16 kSurrogateFirstChar = kTestString[0]; + const char16 kSurrogateSecondChar = kTestString[1]; + string16 result; + + // Elide |kTextString| to all possible widths and check that no instance of + // |kSurrogate| was split in two. + for (int width = 0; width <= kTestStringWidth; width++) { + result = ElideText(kTestString, font, width, TRUNCATE_AT_END); + CheckSurrogatePairs(result, kSurrogateFirstChar, kSurrogateSecondChar); + + result = ElideText(kTestString, font, width, ELIDE_AT_END); + CheckSurrogatePairs(result, kSurrogateFirstChar, kSurrogateSecondChar); + + result = ElideText(kTestString, font, width, ELIDE_IN_MIDDLE); + CheckSurrogatePairs(result, kSurrogateFirstChar, kSurrogateSecondChar); + } +} + +TEST(TextEliderTest, ElideTextLongStrings) { + const string16 kEllipsisStr = UTF8ToUTF16(kEllipsis); + string16 data_scheme(UTF8ToUTF16("data:text/plain,")); + size_t data_scheme_length = data_scheme.length(); + + string16 ten_a(10, 'a'); + string16 hundred_a(100, 'a'); + string16 thousand_a(1000, 'a'); + string16 ten_thousand_a(10000, 'a'); + string16 hundred_thousand_a(100000, 'a'); + string16 million_a(1000000, 'a'); + + size_t number_of_as = 156; + string16 long_string_end( + data_scheme + string16(number_of_as, 'a') + kEllipsisStr); + UTF16Testcase testcases_end[] = { + {data_scheme + ten_a, data_scheme + ten_a}, + {data_scheme + hundred_a, data_scheme + hundred_a}, + {data_scheme + thousand_a, long_string_end}, + {data_scheme + ten_thousand_a, long_string_end}, + {data_scheme + hundred_thousand_a, long_string_end}, + {data_scheme + million_a, long_string_end}, + }; + + const gfx::Font font; + int ellipsis_width = font.GetStringWidth(kEllipsisStr); + for (size_t i = 0; i < arraysize(testcases_end); ++i) { + // Compare sizes rather than actual contents because if the test fails, + // output is rather long. + EXPECT_EQ(testcases_end[i].output.size(), + ElideText(testcases_end[i].input, font, + font.GetStringWidth(testcases_end[i].output), + ELIDE_AT_END).size()); + EXPECT_EQ(kEllipsisStr, + ElideText(testcases_end[i].input, font, ellipsis_width, + ELIDE_AT_END)); + } + + size_t number_of_trailing_as = (data_scheme_length + number_of_as) / 2; + string16 long_string_middle(data_scheme + + string16(number_of_as - number_of_trailing_as, 'a') + kEllipsisStr + + string16(number_of_trailing_as, 'a')); + UTF16Testcase testcases_middle[] = { + {data_scheme + ten_a, data_scheme + ten_a}, + {data_scheme + hundred_a, data_scheme + hundred_a}, + {data_scheme + thousand_a, long_string_middle}, + {data_scheme + ten_thousand_a, long_string_middle}, + {data_scheme + hundred_thousand_a, long_string_middle}, + {data_scheme + million_a, long_string_middle}, + }; + + for (size_t i = 0; i < arraysize(testcases_middle); ++i) { + // Compare sizes rather than actual contents because if the test fails, + // output is rather long. + EXPECT_EQ(testcases_middle[i].output.size(), + ElideText(testcases_middle[i].input, font, + font.GetStringWidth(testcases_middle[i].output), + ELIDE_AT_END).size()); + EXPECT_EQ(kEllipsisStr, + ElideText(testcases_middle[i].input, font, ellipsis_width, + ELIDE_AT_END)); + } +} + +// Verifies display_url is set correctly. +TEST(TextEliderTest, SortedDisplayURL) { + SortedDisplayURL d_url(GURL("http://www.google.com"), std::string()); + EXPECT_EQ("www.google.com", UTF16ToASCII(d_url.display_url())); +} + +// Verifies DisplayURL::Compare works correctly. +TEST(TextEliderTest, SortedDisplayURLCompare) { + UErrorCode create_status = U_ZERO_ERROR; + scoped_ptr<icu::Collator> collator( + icu::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) { + SortedDisplayURL url1(GURL(tests[i].a), std::string()); + SortedDisplayURL url2(GURL(tests[i].b), std::string()); + EXPECT_EQ(tests[i].compare_result, url1.Compare(url2, collator.get())); + EXPECT_EQ(-tests[i].compare_result, url2.Compare(url1, collator.get())); + } +} + +TEST(TextEliderTest, ElideString) { + struct TestData { + const char* input; + int max_len; + bool result; + const char* output; + } cases[] = { + { "Hello", 0, true, "" }, + { "", 0, false, "" }, + { "Hello, my name is Tom", 1, true, "H" }, + { "Hello, my name is Tom", 2, true, "He" }, + { "Hello, my name is Tom", 3, true, "H.m" }, + { "Hello, my name is Tom", 4, true, "H..m" }, + { "Hello, my name is Tom", 5, true, "H...m" }, + { "Hello, my name is Tom", 6, true, "He...m" }, + { "Hello, my name is Tom", 7, true, "He...om" }, + { "Hello, my name is Tom", 10, true, "Hell...Tom" }, + { "Hello, my name is Tom", 100, false, "Hello, my name is Tom" } + }; + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + string16 output; + EXPECT_EQ(cases[i].result, + ElideString(UTF8ToUTF16(cases[i].input), + cases[i].max_len, &output)); + EXPECT_EQ(cases[i].output, UTF16ToUTF8(output)); + } +} + +TEST(TextEliderTest, ElideRectangleText) { + const gfx::Font font; + const int line_height = font.GetHeight(); + const int test_width = font.GetStringWidth(ASCIIToUTF16("Test")); + + struct TestData { + const char* input; + int available_pixel_width; + int available_pixel_height; + bool truncated_y; + const char* output; + } cases[] = { + { "", 0, 0, false, NULL }, + { "", 1, 1, false, NULL }, + { "Test", test_width, 0, true, NULL }, + { "Test", test_width, 1, false, "Test" }, + { "Test", test_width, line_height, false, "Test" }, + { "Test Test", test_width, line_height, true, "Test" }, + { "Test Test", test_width, line_height + 1, false, "Test|Test" }, + { "Test Test", test_width, line_height * 2, false, "Test|Test" }, + { "Test Test", test_width, line_height * 3, false, "Test|Test" }, + { "Test Test", test_width * 2, line_height * 2, false, "Test|Test" }, + { "Test Test", test_width * 3, line_height, false, "Test Test" }, + { "Test\nTest", test_width * 3, line_height * 2, false, "Test|Test" }, + { "Te\nst Te", test_width, line_height * 3, false, "Te|st|Te" }, + { "\nTest", test_width, line_height * 2, false, "|Test" }, + { "\nTest", test_width, line_height, true, "" }, + { "\n\nTest", test_width, line_height * 3, false, "||Test" }, + { "\n\nTest", test_width, line_height * 2, true, "|" }, + { "Test\n", 2 * test_width, line_height * 5, false, "Test|" }, + { "Test\n\n", 2 * test_width, line_height * 5, false, "Test||" }, + { "Test\n\n\n", 2 * test_width, line_height * 5, false, "Test|||" }, + { "Test\nTest\n\n", 2 * test_width, line_height * 5, false, "Test|Test||" }, + { "Test\n\nTest\n", 2 * test_width, line_height * 5, false, "Test||Test|" }, + { "Test\n\n\nTest", 2 * test_width, line_height * 5, false, "Test|||Test" }, + { "Te ", test_width, line_height, false, "Te" }, + { "Te Te Test", test_width, 3 * line_height, false, "Te|Te|Test" }, + }; + + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + std::vector<string16> lines; + EXPECT_EQ(cases[i].truncated_y ? INSUFFICIENT_SPACE_VERTICAL : 0, + ElideRectangleText(UTF8ToUTF16(cases[i].input), + font, + cases[i].available_pixel_width, + cases[i].available_pixel_height, + TRUNCATE_LONG_WORDS, + &lines)); + if (cases[i].output) { + const std::string result = UTF16ToUTF8(JoinString(lines, '|')); + EXPECT_EQ(cases[i].output, result) << "Case " << i << " failed!"; + } else { + EXPECT_TRUE(lines.empty()) << "Case " << i << " failed!"; + } + } +} + +TEST(TextEliderTest, ElideRectangleTextPunctuation) { + const gfx::Font font; + const int line_height = font.GetHeight(); + const int test_width = font.GetStringWidth(ASCIIToUTF16("Test")); + const int test_t_width = font.GetStringWidth(ASCIIToUTF16("Test T")); + + struct TestData { + const char* input; + int available_pixel_width; + int available_pixel_height; + bool wrap_words; + bool truncated_x; + const char* output; + } cases[] = { + { "Test T.", test_t_width, line_height * 2, false, false, "Test|T." }, + { "Test T ?", test_t_width, line_height * 2, false, false, "Test|T ?" }, + { "Test. Test", test_width, line_height * 3, false, true, "Test|Test" }, + { "Test. Test", test_width, line_height * 3, true, false, "Test|.|Test" }, + }; + + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + std::vector<string16> lines; + const WordWrapBehavior wrap_behavior = + (cases[i].wrap_words ? WRAP_LONG_WORDS : TRUNCATE_LONG_WORDS); + EXPECT_EQ(cases[i].truncated_x ? INSUFFICIENT_SPACE_HORIZONTAL : 0, + ElideRectangleText(UTF8ToUTF16(cases[i].input), + font, + cases[i].available_pixel_width, + cases[i].available_pixel_height, + wrap_behavior, + &lines)); + if (cases[i].output) { + const std::string result = UTF16ToUTF8(JoinString(lines, '|')); + EXPECT_EQ(cases[i].output, result) << "Case " << i << " failed!"; + } else { + EXPECT_TRUE(lines.empty()) << "Case " << i << " failed!"; + } + } +} + +TEST(TextEliderTest, ElideRectangleTextLongWords) { + const gfx::Font font; + const int kAvailableHeight = 1000; + const string16 kElidedTesting = UTF8ToUTF16(std::string("Tes") + kEllipsis); + const int elided_width = font.GetStringWidth(kElidedTesting); + const int test_width = font.GetStringWidth(ASCIIToUTF16("Test")); + + struct TestData { + const char* input; + int available_pixel_width; + WordWrapBehavior wrap_behavior; + bool truncated_x; + const char* output; + } cases[] = { + { "Testing", test_width, IGNORE_LONG_WORDS, false, "Testing" }, + { "X Testing", test_width, IGNORE_LONG_WORDS, false, "X|Testing" }, + { "Test Testing", test_width, IGNORE_LONG_WORDS, false, "Test|Testing" }, + { "Test\nTesting", test_width, IGNORE_LONG_WORDS, false, "Test|Testing" }, + { "Test Tests ", test_width, IGNORE_LONG_WORDS, false, "Test|Tests" }, + { "Test Tests T", test_width, IGNORE_LONG_WORDS, false, "Test|Tests|T" }, + + { "Testing", elided_width, ELIDE_LONG_WORDS, true, "Tes..." }, + { "X Testing", elided_width, ELIDE_LONG_WORDS, true, "X|Tes..." }, + { "Test Testing", elided_width, ELIDE_LONG_WORDS, true, "Test|Tes..." }, + { "Test\nTesting", elided_width, ELIDE_LONG_WORDS, true, "Test|Tes..." }, + + { "Testing", test_width, TRUNCATE_LONG_WORDS, true, "Test" }, + { "X Testing", test_width, TRUNCATE_LONG_WORDS, true, "X|Test" }, + { "Test Testing", test_width, TRUNCATE_LONG_WORDS, true, "Test|Test" }, + { "Test\nTesting", test_width, TRUNCATE_LONG_WORDS, true, "Test|Test" }, + { "Test Tests ", test_width, TRUNCATE_LONG_WORDS, true, "Test|Test" }, + { "Test Tests T", test_width, TRUNCATE_LONG_WORDS, true, "Test|Test|T" }, + + { "Testing", test_width, WRAP_LONG_WORDS, false, "Test|ing" }, + { "X Testing", test_width, WRAP_LONG_WORDS, false, "X|Test|ing" }, + { "Test Testing", test_width, WRAP_LONG_WORDS, false, "Test|Test|ing" }, + { "Test\nTesting", test_width, WRAP_LONG_WORDS, false, "Test|Test|ing" }, + { "Test Tests ", test_width, WRAP_LONG_WORDS, false, "Test|Test|s" }, + { "Test Tests T", test_width, WRAP_LONG_WORDS, false, "Test|Test|s T" }, + { "TestTestTest", test_width, WRAP_LONG_WORDS, false, "Test|Test|Test" }, + { "TestTestTestT", test_width, WRAP_LONG_WORDS, false, "Test|Test|Test|T" }, + }; + + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + std::vector<string16> lines; + EXPECT_EQ(cases[i].truncated_x ? INSUFFICIENT_SPACE_HORIZONTAL : 0, + ElideRectangleText(UTF8ToUTF16(cases[i].input), + font, + cases[i].available_pixel_width, + kAvailableHeight, + cases[i].wrap_behavior, + &lines)); + std::string expected_output(cases[i].output); + ReplaceSubstringsAfterOffset(&expected_output, 0, "...", kEllipsis); + const std::string result = UTF16ToUTF8(JoinString(lines, '|')); + EXPECT_EQ(expected_output, result) << "Case " << i << " failed!"; + } +} + +TEST(TextEliderTest, ElideRectangleString) { + struct TestData { + const char* input; + int max_rows; + int max_cols; + bool result; + const char* output; + } cases[] = { + { "", 0, 0, false, "" }, + { "", 1, 1, false, "" }, + { "Hi, my name is\nTom", 0, 0, true, "..." }, + { "Hi, my name is\nTom", 1, 0, true, "\n..." }, + { "Hi, my name is\nTom", 0, 1, true, "..." }, + { "Hi, my name is\nTom", 1, 1, true, "H\n..." }, + { "Hi, my name is\nTom", 2, 1, true, "H\ni\n..." }, + { "Hi, my name is\nTom", 3, 1, true, "H\ni\n,\n..." }, + { "Hi, my name is\nTom", 4, 1, true, "H\ni\n,\n \n..." }, + { "Hi, my name is\nTom", 5, 1, true, "H\ni\n,\n \nm\n..." }, + { "Hi, my name is\nTom", 0, 2, true, "..." }, + { "Hi, my name is\nTom", 1, 2, true, "Hi\n..." }, + { "Hi, my name is\nTom", 2, 2, true, "Hi\n, \n..." }, + { "Hi, my name is\nTom", 3, 2, true, "Hi\n, \nmy\n..." }, + { "Hi, my name is\nTom", 4, 2, true, "Hi\n, \nmy\n n\n..." }, + { "Hi, my name is\nTom", 5, 2, true, "Hi\n, \nmy\n n\nam\n..." }, + { "Hi, my name is\nTom", 0, 3, true, "..." }, + { "Hi, my name is\nTom", 1, 3, true, "Hi,\n..." }, + { "Hi, my name is\nTom", 2, 3, true, "Hi,\n my\n..." }, + { "Hi, my name is\nTom", 3, 3, true, "Hi,\n my\n na\n..." }, + { "Hi, my name is\nTom", 4, 3, true, "Hi,\n my\n na\nme \n..." }, + { "Hi, my name is\nTom", 5, 3, true, "Hi,\n my\n na\nme \nis\n..." }, + { "Hi, my name is\nTom", 1, 4, true, "Hi, \n..." }, + { "Hi, my name is\nTom", 2, 4, true, "Hi, \nmy n\n..." }, + { "Hi, my name is\nTom", 3, 4, true, "Hi, \nmy n\name \n..." }, + { "Hi, my name is\nTom", 4, 4, true, "Hi, \nmy n\name \nis\n..." }, + { "Hi, my name is\nTom", 5, 4, false, "Hi, \nmy n\name \nis\nTom" }, + { "Hi, my name is\nTom", 1, 5, true, "Hi, \n..." }, + { "Hi, my name is\nTom", 2, 5, true, "Hi, \nmy na\n..." }, + { "Hi, my name is\nTom", 3, 5, true, "Hi, \nmy na\nme \n..." }, + { "Hi, my name is\nTom", 4, 5, true, "Hi, \nmy na\nme \nis\n..." }, + { "Hi, my name is\nTom", 5, 5, false, "Hi, \nmy na\nme \nis\nTom" }, + { "Hi, my name is\nTom", 1, 6, true, "Hi, \n..." }, + { "Hi, my name is\nTom", 2, 6, true, "Hi, \nmy \n..." }, + { "Hi, my name is\nTom", 3, 6, true, "Hi, \nmy \nname \n..." }, + { "Hi, my name is\nTom", 4, 6, true, "Hi, \nmy \nname \nis\n..." }, + { "Hi, my name is\nTom", 5, 6, false, "Hi, \nmy \nname \nis\nTom" }, + { "Hi, my name is\nTom", 1, 7, true, "Hi, \n..." }, + { "Hi, my name is\nTom", 2, 7, true, "Hi, \nmy \n..." }, + { "Hi, my name is\nTom", 3, 7, true, "Hi, \nmy \nname \n..." }, + { "Hi, my name is\nTom", 4, 7, true, "Hi, \nmy \nname \nis\n..." }, + { "Hi, my name is\nTom", 5, 7, false, "Hi, \nmy \nname \nis\nTom" }, + { "Hi, my name is\nTom", 1, 8, true, "Hi, my \n..." }, + { "Hi, my name is\nTom", 2, 8, true, "Hi, my \nname \n..." }, + { "Hi, my name is\nTom", 3, 8, true, "Hi, my \nname \nis\n..." }, + { "Hi, my name is\nTom", 4, 8, false, "Hi, my \nname \nis\nTom" }, + { "Hi, my name is\nTom", 1, 9, true, "Hi, my \n..." }, + { "Hi, my name is\nTom", 2, 9, true, "Hi, my \nname is\n..." }, + { "Hi, my name is\nTom", 3, 9, false, "Hi, my \nname is\nTom" }, + { "Hi, my name is\nTom", 1, 10, true, "Hi, my \n..." }, + { "Hi, my name is\nTom", 2, 10, true, "Hi, my \nname is\n..." }, + { "Hi, my name is\nTom", 3, 10, false, "Hi, my \nname is\nTom" }, + { "Hi, my name is\nTom", 1, 11, true, "Hi, my \n..." }, + { "Hi, my name is\nTom", 2, 11, true, "Hi, my \nname is\n..." }, + { "Hi, my name is\nTom", 3, 11, false, "Hi, my \nname is\nTom" }, + { "Hi, my name is\nTom", 1, 12, true, "Hi, my \n..." }, + { "Hi, my name is\nTom", 2, 12, true, "Hi, my \nname is\n..." }, + { "Hi, my name is\nTom", 3, 12, false, "Hi, my \nname is\nTom" }, + { "Hi, my name is\nTom", 1, 13, true, "Hi, my name \n..." }, + { "Hi, my name is\nTom", 2, 13, true, "Hi, my name \nis\n..." }, + { "Hi, my name is\nTom", 3, 13, false, "Hi, my name \nis\nTom" }, + { "Hi, my name is\nTom", 1, 20, true, "Hi, my name is\n..." }, + { "Hi, my name is\nTom", 2, 20, false, "Hi, my name is\nTom" }, + { "Hi, my name is Tom", 1, 40, false, "Hi, my name is Tom" }, + }; + string16 output; + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + EXPECT_EQ(cases[i].result, + ElideRectangleString(UTF8ToUTF16(cases[i].input), + cases[i].max_rows, cases[i].max_cols, + true, &output)); + EXPECT_EQ(cases[i].output, UTF16ToUTF8(output)); + } +} + +TEST(TextEliderTest, ElideRectangleStringNotStrict) { + struct TestData { + const char* input; + int max_rows; + int max_cols; + bool result; + const char* output; + } cases[] = { + { "", 0, 0, false, "" }, + { "", 1, 1, false, "" }, + { "Hi, my name_is\nDick", 0, 0, true, "..." }, + { "Hi, my name_is\nDick", 1, 0, true, "\n..." }, + { "Hi, my name_is\nDick", 0, 1, true, "..." }, + { "Hi, my name_is\nDick", 1, 1, true, "H\n..." }, + { "Hi, my name_is\nDick", 2, 1, true, "H\ni\n..." }, + { "Hi, my name_is\nDick", 3, 1, true, "H\ni\n,\n..." }, + { "Hi, my name_is\nDick", 4, 1, true, "H\ni\n,\n \n..." }, + { "Hi, my name_is\nDick", 5, 1, true, "H\ni\n,\n \nm\n..." }, + { "Hi, my name_is\nDick", 0, 2, true, "..." }, + { "Hi, my name_is\nDick", 1, 2, true, "Hi\n..." }, + { "Hi, my name_is\nDick", 2, 2, true, "Hi\n, \n..." }, + { "Hi, my name_is\nDick", 3, 2, true, "Hi\n, \nmy\n..." }, + { "Hi, my name_is\nDick", 4, 2, true, "Hi\n, \nmy\n n\n..." }, + { "Hi, my name_is\nDick", 5, 2, true, "Hi\n, \nmy\n n\nam\n..." }, + { "Hi, my name_is\nDick", 0, 3, true, "..." }, + { "Hi, my name_is\nDick", 1, 3, true, "Hi,\n..." }, + { "Hi, my name_is\nDick", 2, 3, true, "Hi,\n my\n..." }, + { "Hi, my name_is\nDick", 3, 3, true, "Hi,\n my\n na\n..." }, + { "Hi, my name_is\nDick", 4, 3, true, "Hi,\n my\n na\nme_\n..." }, + { "Hi, my name_is\nDick", 5, 3, true, "Hi,\n my\n na\nme_\nis\n..." }, + { "Hi, my name_is\nDick", 1, 4, true, "Hi, ..." }, + { "Hi, my name_is\nDick", 2, 4, true, "Hi, my n\n..." }, + { "Hi, my name_is\nDick", 3, 4, true, "Hi, my n\name_\n..." }, + { "Hi, my name_is\nDick", 4, 4, true, "Hi, my n\name_\nis\n..." }, + { "Hi, my name_is\nDick", 5, 4, false, "Hi, my n\name_\nis\nDick" }, + { "Hi, my name_is\nDick", 1, 5, true, "Hi, ..." }, + { "Hi, my name_is\nDick", 2, 5, true, "Hi, my na\n..." }, + { "Hi, my name_is\nDick", 3, 5, true, "Hi, my na\nme_is\n..." }, + { "Hi, my name_is\nDick", 4, 5, true, "Hi, my na\nme_is\n\n..." }, + { "Hi, my name_is\nDick", 5, 5, false, "Hi, my na\nme_is\n\nDick" }, + { "Hi, my name_is\nDick", 1, 6, true, "Hi, ..." }, + { "Hi, my name_is\nDick", 2, 6, true, "Hi, my nam\n..." }, + { "Hi, my name_is\nDick", 3, 6, true, "Hi, my nam\ne_is\n..." }, + { "Hi, my name_is\nDick", 4, 6, false, "Hi, my nam\ne_is\nDick" }, + { "Hi, my name_is\nDick", 5, 6, false, "Hi, my nam\ne_is\nDick" }, + { "Hi, my name_is\nDick", 1, 7, true, "Hi, ..." }, + { "Hi, my name_is\nDick", 2, 7, true, "Hi, my name\n..." }, + { "Hi, my name_is\nDick", 3, 7, true, "Hi, my name\n_is\n..." }, + { "Hi, my name_is\nDick", 4, 7, false, "Hi, my name\n_is\nDick" }, + { "Hi, my name_is\nDick", 5, 7, false, "Hi, my name\n_is\nDick" }, + { "Hi, my name_is\nDick", 1, 8, true, "Hi, my n\n..." }, + { "Hi, my name_is\nDick", 2, 8, true, "Hi, my n\name_is\n..." }, + { "Hi, my name_is\nDick", 3, 8, false, "Hi, my n\name_is\nDick" }, + { "Hi, my name_is\nDick", 1, 9, true, "Hi, my ..." }, + { "Hi, my name_is\nDick", 2, 9, true, "Hi, my name_is\n..." }, + { "Hi, my name_is\nDick", 3, 9, false, "Hi, my name_is\nDick" }, + { "Hi, my name_is\nDick", 1, 10, true, "Hi, my ..." }, + { "Hi, my name_is\nDick", 2, 10, true, "Hi, my name_is\n..." }, + { "Hi, my name_is\nDick", 3, 10, false, "Hi, my name_is\nDick" }, + { "Hi, my name_is\nDick", 1, 11, true, "Hi, my ..." }, + { "Hi, my name_is\nDick", 2, 11, true, "Hi, my name_is\n..." }, + { "Hi, my name_is\nDick", 3, 11, false, "Hi, my name_is\nDick" }, + { "Hi, my name_is\nDick", 1, 12, true, "Hi, my ..." }, + { "Hi, my name_is\nDick", 2, 12, true, "Hi, my name_is\n..." }, + { "Hi, my name_is\nDick", 3, 12, false, "Hi, my name_is\nDick" }, + { "Hi, my name_is\nDick", 1, 13, true, "Hi, my ..." }, + { "Hi, my name_is\nDick", 2, 13, true, "Hi, my name_is\n..." }, + { "Hi, my name_is\nDick", 3, 13, false, "Hi, my name_is\nDick" }, + { "Hi, my name_is\nDick", 1, 20, true, "Hi, my name_is\n..." }, + { "Hi, my name_is\nDick", 2, 20, false, "Hi, my name_is\nDick" }, + { "Hi, my name_is Dick", 1, 40, false, "Hi, my name_is Dick" }, + }; + string16 output; + for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); ++i) { + EXPECT_EQ(cases[i].result, + ElideRectangleString(UTF8ToUTF16(cases[i].input), + cases[i].max_rows, cases[i].max_cols, + false, &output)); + EXPECT_EQ(cases[i].output, UTF16ToUTF8(output)); + } +} + +TEST(TextEliderTest, ElideRectangleWide16) { + // Two greek words separated by space. + const string16 str(WideToUTF16( + L"\x03a0\x03b1\x03b3\x03ba\x03cc\x03c3\x03bc\x03b9" + L"\x03bf\x03c2\x0020\x0399\x03c3\x03c4\x03cc\x03c2")); + const string16 out1(WideToUTF16( + L"\x03a0\x03b1\x03b3\x03ba\n" + L"\x03cc\x03c3\x03bc\x03b9\n" + L"...")); + const string16 out2(WideToUTF16( + L"\x03a0\x03b1\x03b3\x03ba\x03cc\x03c3\x03bc\x03b9\x03bf\x03c2\x0020\n" + L"\x0399\x03c3\x03c4\x03cc\x03c2")); + string16 output; + EXPECT_TRUE(ElideRectangleString(str, 2, 4, true, &output)); + EXPECT_EQ(out1, output); + EXPECT_FALSE(ElideRectangleString(str, 2, 12, true, &output)); + EXPECT_EQ(out2, output); +} + +TEST(TextEliderTest, ElideRectangleWide32) { + // Four U+1D49C MATHEMATICAL SCRIPT CAPITAL A followed by space "aaaaa". + const string16 str(UTF8ToUTF16( + "\xF0\x9D\x92\x9C\xF0\x9D\x92\x9C\xF0\x9D\x92\x9C\xF0\x9D\x92\x9C" + " aaaaa")); + const string16 out(UTF8ToUTF16( + "\xF0\x9D\x92\x9C\xF0\x9D\x92\x9C\xF0\x9D\x92\x9C\n" + "\xF0\x9D\x92\x9C \naaa\n...")); + string16 output; + EXPECT_TRUE(ElideRectangleString(str, 3, 3, true, &output)); + EXPECT_EQ(out, output); +} + +TEST(TextEliderTest, TruncateString) { + string16 string = ASCIIToUTF16("foooooey bxxxar baz"); + + // Make sure it doesn't modify the string if length > string length. + EXPECT_EQ(string, TruncateString(string, 100)); + + // Test no characters. + EXPECT_EQ(L"", UTF16ToWide(TruncateString(string, 0))); + + // Test 1 character. + EXPECT_EQ(L"\x2026", UTF16ToWide(TruncateString(string, 1))); + + // Test adds ... at right spot when there is enough room to break at a + // word boundary. + EXPECT_EQ(L"foooooey\x2026", UTF16ToWide(TruncateString(string, 14))); + + // Test adds ... at right spot when there is not enough space in first word. + EXPECT_EQ(L"f\x2026", UTF16ToWide(TruncateString(string, 2))); + + // Test adds ... at right spot when there is not enough room to break at a + // word boundary. + EXPECT_EQ(L"foooooey\x2026", UTF16ToWide(TruncateString(string, 11))); + + // Test completely truncates string if break is on initial whitespace. + EXPECT_EQ(L"\x2026", UTF16ToWide(TruncateString(ASCIIToUTF16(" "), 2))); +} + +} // namespace gfx diff --git a/ui/message_center/cocoa/notification_controller.mm b/ui/message_center/cocoa/notification_controller.mm index ab01878..59af207 100644 --- a/ui/message_center/cocoa/notification_controller.mm +++ b/ui/message_center/cocoa/notification_controller.mm @@ -14,7 +14,7 @@ #import "ui/base/cocoa/hover_image_button.h" #include "ui/base/l10n/l10n_util_mac.h" #include "ui/base/resource/resource_bundle.h" -#include "ui/base/text/text_elider.h" +#include "ui/gfx/text_elider.h" #include "ui/message_center/message_center.h" #include "ui/message_center/message_center_style.h" #include "ui/message_center/notification.h" @@ -697,15 +697,15 @@ int height = (lines + 1) * font.GetHeight(); std::vector<string16> wrapped; - ui::ElideRectangleText(text, font, width, height, - ui::WRAP_LONG_WORDS, &wrapped); + gfx::ElideRectangleText(text, font, width, height, + gfx::WRAP_LONG_WORDS, &wrapped); if (wrapped.size() > lines) { // Add an ellipsis to the last line. If this ellipsis makes the last line - // too wide, that line will be further elided by the ui::ElideText below. - string16 last = wrapped[lines - 1] + UTF8ToUTF16(ui::kEllipsis); + // too wide, that line will be further elided by the gfx::ElideText below. + string16 last = wrapped[lines - 1] + UTF8ToUTF16(gfx::kEllipsis); if (font.GetStringWidth(last) > width) - last = ui::ElideText(last, font, width, ui::ELIDE_AT_END); + last = gfx::ElideText(last, font, width, gfx::ELIDE_AT_END); wrapped.resize(lines - 1); wrapped.push_back(last); } diff --git a/ui/message_center/views/bounded_label.cc b/ui/message_center/views/bounded_label.cc index 6752795..1fe428a 100644 --- a/ui/message_center/views/bounded_label.cc +++ b/ui/message_center/views/bounded_label.cc @@ -8,8 +8,8 @@ #include "base/strings/string_util.h" #include "base/strings/utf_string_conversions.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/canvas.h" +#include "ui/gfx/text_elider.h" #include "ui/views/controls/label.h" namespace { @@ -143,18 +143,18 @@ std::vector<string16> InnerBoundedLabel::GetWrappedText(int width, int lines) { // Wrap, using INT_MAX for -1 widths that indicate no wrapping. std::vector<string16> wrapped; - ui::ElideRectangleText(text(), font(), + gfx::ElideRectangleText(text(), font(), (width < 0) ? std::numeric_limits<int>::max() : width, - height, ui::WRAP_LONG_WORDS, &wrapped); + height, gfx::WRAP_LONG_WORDS, &wrapped); // Elide if necessary. if (lines > 0 && wrapped.size() > static_cast<unsigned int>(lines)) { // Add an ellipsis to the last line. If this ellipsis makes the last line - // too wide, that line will be further elided by the ui::ElideText below, + // too wide, that line will be further elided by the gfx::ElideText below, // so for example "ABC" could become "ABC..." and then "AB...". - string16 last = wrapped[lines - 1] + UTF8ToUTF16(ui::kEllipsis); + string16 last = wrapped[lines - 1] + UTF8ToUTF16(gfx::kEllipsis); if (width > 0 && font().GetStringWidth(last) > width) - last = ui::ElideText(last, font(), width, ui::ELIDE_AT_END); + last = gfx::ElideText(last, font(), width, gfx::ELIDE_AT_END); wrapped.resize(lines - 1); wrapped.push_back(last); } diff --git a/ui/message_center/views/notification_view.cc b/ui/message_center/views/notification_view.cc index a39b4e4..5d878a3 100644 --- a/ui/message_center/views/notification_view.cc +++ b/ui/message_center/views/notification_view.cc @@ -10,10 +10,10 @@ #include "grit/ui_resources.h" #include "ui/base/layout.h" #include "ui/base/resource/resource_bundle.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/canvas.h" #include "ui/gfx/size.h" #include "ui/gfx/skia_util.h" +#include "ui/gfx/text_elider.h" #include "ui/message_center/message_center.h" #include "ui/message_center/message_center_style.h" #include "ui/message_center/message_center_switches.h" @@ -472,7 +472,7 @@ NotificationView::NotificationView(const Notification& notification, gfx::Font font = views::Label().font().DeriveFont(2); int padding = kTitleLineHeight - font.GetHeight(); title_view_ = new BoundedLabel( - ui::TruncateString(notification.title(), kTitleCharacterLimit), font); + gfx::TruncateString(notification.title(), kTitleCharacterLimit), font); title_view_->SetLineHeight(kTitleLineHeight); title_view_->SetLineLimit(message_center::kTitleLineLimit); title_view_->SetColors(message_center::kRegularTextColor, @@ -487,7 +487,7 @@ NotificationView::NotificationView(const Notification& notification, if (!notification.message().empty()) { int padding = kMessageLineHeight - views::Label().font().GetHeight(); message_view_ = new BoundedLabel( - ui::TruncateString(notification.message(), kMessageCharacterLimit)); + gfx::TruncateString(notification.message(), kMessageCharacterLimit)); message_view_->SetLineHeight(kMessageLineHeight); message_view_->SetVisible(!is_expanded() || !notification.items().size()); message_view_->SetColors(message_center::kRegularTextColor, @@ -503,7 +503,7 @@ NotificationView::NotificationView(const Notification& notification, gfx::Font font = views::Label().font(); int padding = kMessageLineHeight - font.GetHeight(); context_message_view_ = - new BoundedLabel(ui::TruncateString(notification.context_message(), + new BoundedLabel(gfx::TruncateString(notification.context_message(), kContextMessageCharacterLimit), font); context_message_view_->SetLineLimit( @@ -316,8 +316,6 @@ 'base/resource/resource_handle.h', 'base/text/bytes_formatting.cc', 'base/text/bytes_formatting.h', - 'base/text/text_elider.cc', - 'base/text/text_elider.h', 'base/text/utf16_indexing.cc', 'base/text/utf16_indexing.h', 'base/theme_provider.cc', @@ -583,6 +581,8 @@ 'gfx/sys_color_change_listener.cc', 'gfx/sys_color_change_listener.h', 'gfx/text_constants.h', + 'gfx/text_elider.cc', + 'gfx/text_elider.h', 'gfx/text_utils.cc', 'gfx/text_utils.h', 'gfx/text_utils_android.cc', diff --git a/ui/ui_unittests.gypi b/ui/ui_unittests.gypi index 38c72e1..8c9771b 100644 --- a/ui/ui_unittests.gypi +++ b/ui/ui_unittests.gypi @@ -104,7 +104,6 @@ 'base/resource/data_pack_literal.cc', 'base/resource/data_pack_unittest.cc', 'base/resource/resource_bundle_unittest.cc', - 'base/text/text_elider_unittest.cc', 'gfx/box_unittest.cc', 'gfx/codec/png_codec_unittest.cc', 'gfx/color_utils_unittest.cc', @@ -131,6 +130,7 @@ 'gfx/shadow_value_unittest.cc', 'gfx/size_unittest.cc', 'gfx/skbitmap_operations_unittest.cc', + 'gfx/text_elider_unittest.cc', 'gfx/text_utils_unittest.cc', 'gfx/vector2d_unittest.cc', 'gfx/vector3d_unittest.cc', diff --git a/ui/views/controls/label.cc b/ui/views/controls/label.cc index 05f9aed..7d4bc7e 100644 --- a/ui/views/controls/label.cc +++ b/ui/views/controls/label.cc @@ -16,11 +16,11 @@ #include "base/strings/utf_string_conversions.h" #include "ui/base/accessibility/accessible_view_state.h" #include "ui/base/resource/resource_bundle.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/canvas.h" #include "ui/gfx/color_utils.h" #include "ui/gfx/insets.h" #include "ui/gfx/shadow_value.h" +#include "ui/gfx/text_elider.h" #include "ui/gfx/text_utils.h" #include "ui/native_theme/native_theme.h" #include "ui/views/background.h" @@ -495,14 +495,15 @@ void Label::CalculateDrawStringParams(string16* paint_text, if (is_multi_line_ || (elide_behavior_ == NO_ELIDE)) { *paint_text = text_; } else if (elide_behavior_ == ELIDE_IN_MIDDLE) { - *paint_text = ui::ElideText(text_, font_list_, GetAvailableRect().width(), - ui::ELIDE_IN_MIDDLE); + *paint_text = gfx::ElideText(text_, font_list_, GetAvailableRect().width(), + gfx::ELIDE_IN_MIDDLE); } else if (elide_behavior_ == ELIDE_AT_END) { - *paint_text = ui::ElideText(text_, font_list_, GetAvailableRect().width(), - ui::ELIDE_AT_END); + *paint_text = gfx::ElideText(text_, font_list_, GetAvailableRect().width(), + gfx::ELIDE_AT_END); } else { DCHECK_EQ(ELIDE_AS_EMAIL, elide_behavior_); - *paint_text = ui::ElideEmail(text_, font_list_, GetAvailableRect().width()); + *paint_text = gfx::ElideEmail(text_, font_list_, + GetAvailableRect().width()); } *text_bounds = GetTextBounds(); diff --git a/ui/views/controls/styled_label.cc b/ui/views/controls/styled_label.cc index 268f3db..735c8ee 100644 --- a/ui/views/controls/styled_label.cc +++ b/ui/views/controls/styled_label.cc @@ -7,7 +7,7 @@ #include <vector> #include "base/strings/string_util.h" -#include "ui/base/text/text_elider.h" +#include "ui/gfx/text_elider.h" #include "ui/native_theme/native_theme.h" #include "ui/views/controls/label.h" #include "ui/views/controls/link.h" @@ -188,11 +188,11 @@ int StyledLabel::CalculateAndDoLayout(int width, bool dry_run) { text_font = text_font.DeriveFont(0, style_ranges.top().style_info.font_style); } - ui::ElideRectangleText(remaining_string, + gfx::ElideRectangleText(remaining_string, text_font, chunk_bounds.width(), chunk_bounds.height(), - ui::IGNORE_LONG_WORDS, + gfx::IGNORE_LONG_WORDS, &substrings); DCHECK(!substrings.empty()); diff --git a/ui/views/corewm/tooltip_controller.cc b/ui/views/corewm/tooltip_controller.cc index 1c17d91..f9943df 100644 --- a/ui/views/corewm/tooltip_controller.cc +++ b/ui/views/corewm/tooltip_controller.cc @@ -17,11 +17,11 @@ #include "ui/aura/window.h" #include "ui/base/events/event.h" #include "ui/base/resource/resource_bundle.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/font.h" #include "ui/gfx/point.h" #include "ui/gfx/rect.h" #include "ui/gfx/screen.h" +#include "ui/gfx/text_elider.h" #include "ui/views/background.h" #include "ui/views/border.h" #include "ui/views/controls/label.h" @@ -407,7 +407,7 @@ void TooltipController::TrimTooltipToFit(int max_width, if (result_lines.size() > kMaxLines) { result_lines.resize(kMaxLines); // Add ellipses character to last line. - result_lines[kMaxLines - 1] = ui::TruncateString( + result_lines[kMaxLines - 1] = gfx::TruncateString( result_lines.back(), result_lines.back().length() - 1); } *line_count = result_lines.size(); @@ -424,7 +424,8 @@ void TooltipController::TrimTooltipToFit(int max_width, // case, we simply truncate at available_width and add ellipses at the end. if (line_width > available_width) { *width = available_width; - result.append(ui::ElideText(*l, font, available_width, ui::ELIDE_AT_END)); + result.append(gfx::ElideText(*l, font, available_width, + gfx::ELIDE_AT_END)); } else { *width = std::max(*width, line_width); result.append(*l); diff --git a/ui/views/corewm/tooltip_controller_unittest.cc b/ui/views/corewm/tooltip_controller_unittest.cc index 2ea2129..61dfb18 100644 --- a/ui/views/corewm/tooltip_controller_unittest.cc +++ b/ui/views/corewm/tooltip_controller_unittest.cc @@ -13,9 +13,9 @@ #include "ui/aura/test/event_generator.h" #include "ui/aura/window.h" #include "ui/base/resource/resource_bundle.h" -#include "ui/base/text/text_elider.h" #include "ui/gfx/font.h" #include "ui/gfx/point.h" +#include "ui/gfx/text_elider.h" #include "ui/views/corewm/tooltip_controller_test_helper.h" #include "ui/views/view.h" #include "ui/views/widget/widget.h" @@ -274,8 +274,8 @@ TEST_F(TooltipControllerTest, TrimTooltipToFitTests) { max_width, &tooltip, &width, &line_count); EXPECT_NEAR(max_pixel_width, width, 5); EXPECT_EQ(1, line_count); - EXPECT_EQ(ui::ElideText(UTF8ToUTF16(std::string('a', max_pixel_width)), font, - max_pixel_width, ui::ELIDE_AT_END), tooltip); + EXPECT_EQ(gfx::ElideText(UTF8ToUTF16(std::string('a', max_pixel_width)), font, + max_pixel_width, gfx::ELIDE_AT_END), tooltip); #endif // Normal small tooltip should stay as is. diff --git a/ui/views/widget/tooltip_manager.cc b/ui/views/widget/tooltip_manager.cc index af51f0d..cbfba8f 100644 --- a/ui/views/widget/tooltip_manager.cc +++ b/ui/views/widget/tooltip_manager.cc @@ -8,7 +8,7 @@ #include "base/strings/string_split.h" #include "base/strings/utf_string_conversions.h" -#include "ui/base/text/text_elider.h" +#include "ui/gfx/text_elider.h" #include "ui/gfx/text_utils.h" // Maximum number of characters we allow in a tooltip. @@ -50,7 +50,7 @@ void TooltipManager::TrimTooltipToFit(string16* text, for (std::vector<string16>::iterator i = lines.begin(); i != lines.end(); ++i) { string16 elided_text = - ui::ElideText(*i, font_list, available_width, ui::ELIDE_AT_END); + gfx::ElideText(*i, font_list, available_width, gfx::ELIDE_AT_END); *max_width = std::max(*max_width, gfx::GetStringWidth(elided_text, font_list)); if (!result.empty()) |