Chromium Code Reviews| Index: ui/base/text/text_elider.cc |
| diff --git a/ui/base/text/text_elider.cc b/ui/base/text/text_elider.cc |
| index 525cea0df61eb373507b91372f144dbf327cf419..08deea618d57ce0717a149455f0880571bfcc20b 100644 |
| --- a/ui/base/text/text_elider.cc |
| +++ b/ui/base/text/text_elider.cc |
| @@ -171,8 +171,8 @@ string16 ElideUrl(const GURL& url, |
| if (available_pixel_width <= 0) |
| return url_string; |
| - // If non-standard or not file type, return plain eliding. |
| - if (!(url.SchemeIsFile() || url.IsStandard())) |
|
sky
2012/02/15 16:17:28
nit: I prefer !foo && !foo. Easier to parse.
ericu
2012/02/15 22:32:10
Actually, you're commenting on the "before" code,
|
| + // If non-standard, return plain eliding. |
| + if (!url.IsStandard()) |
| return ElideText(url_string, font, available_pixel_width, ELIDE_AT_END); |
| // Now start eliding url_string to fit within available pixel width. |