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

Unified Diff: ui/gfx/font_list.cc

Issue 24883002: Uses and returns the fractional width in text eliding (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix trybots Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/gfx/font_list.cc
diff --git a/ui/gfx/font_list.cc b/ui/gfx/font_list.cc
index 72c877754e59bfeeba9bf63af3ad8f652385d7af..cd8abc173af2cc2478f9019e16eecdd3f0932725 100644
--- a/ui/gfx/font_list.cc
+++ b/ui/gfx/font_list.cc
@@ -195,6 +195,13 @@ int FontList::GetStringWidth(const base::string16& text) const {
return GetPrimaryFont().GetStringWidth(text);
}
+float FontList::GetStringWidthF(const base::string16& text) const {
Alexei Svitkine (slow) 2013/10/07 21:43:39 Do we need this? Or can everything that needs it u
jianli 2013/10/07 23:18:45 Done.
+ // Rely on the primary font metrics for the time being.
+ // TODO(yukishiino): Not only the first font, all the fonts in the list should
+ // be taken into account to compute the pixels needed to display |text|.
+ return GetPrimaryFont().GetStringWidthF(text);
+}
+
int FontList::GetExpectedTextWidth(int length) const {
// Rely on the primary font metrics for the time being.
return GetPrimaryFont().GetExpectedTextWidth(length);

Powered by Google App Engine
This is Rietveld 408576698