Skip to content
Snippets Groups Projects
Commit 66332e24 authored by pkasting@chromium.org's avatar pkasting@chromium.org
Browse files

Not all text-sizing calls took the insets into account, leading to inconsistent eliding.

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/4233003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@64830 0039d316-1c4b-4281-b951-d872f2087c98
parent 3d0d8d4e
No related branches found
No related tags found
No related merge requests found
......@@ -184,7 +184,8 @@ bool Label::GetTooltipText(const gfx::Point& p, std::wstring* tooltip) {
}
// Show the full text if the text does not fit.
if (!is_multi_line_ && font_.GetStringWidth(text_) > width()) {
if (!is_multi_line_ &&
(font_.GetStringWidth(text_) > GetAvailableRect().width())) {
*tooltip = text_;
return true;
}
......@@ -423,8 +424,8 @@ void Label::CalculateDrawStringParams(std::wstring* paint_text,
if (url_set_) {
// TODO(jungshik) : Figure out how to get 'intl.accept_languages'
// preference and use it when calling ElideUrl.
*paint_text = UTF16ToWideHack(gfx::ElideUrl(url_, font_, width(),
std::wstring()));
*paint_text = UTF16ToWideHack(
gfx::ElideUrl(url_, font_, GetAvailableRect().width(), std::wstring()));
// An URLs is always treated as an LTR text and therefore we should
// explicitly mark it as such if the locale is RTL so that URLs containing
......@@ -439,7 +440,7 @@ void Label::CalculateDrawStringParams(std::wstring* paint_text,
WideToUTF16(*paint_text)));
} else if (elide_in_middle_) {
*paint_text = UTF16ToWideHack(gfx::ElideText(WideToUTF16Hack(text_),
font_, width(), true));
font_, GetAvailableRect().width(), true));
} else {
*paint_text = text_;
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment