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

Issue 3010031: [Mac] Elide long Omnibox contents. (Closed)

Created:
10 years, 5 months ago by Scott Hess - ex-Googler
Modified:
9 years, 6 months ago
Reviewers:
Miranda Callahan
CC:
chromium-reviews, John Grabowski, pam+watch_chromium.org, ben+cc_chromium.org
Base URL:
git://codf21.jail/chromium.git
Visibility:
Public.

Description

[Mac] Elide long Omnibox contents. Doing the most simple eliding, which is to truncate and replace with ellipses on a character basis. I put this in the code rather than the .xib because this seems like it should be a hard-wired Omnibox behavior. BUG=50253 TEST=Put in a very long URL and check that it elides as the window gets too small for it to fit. You can easily craft long URLs by adding # and filling with whatever you desire after that. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53826

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M chrome/browser/cocoa/location_bar/autocomplete_text_field.mm View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Scott Hess - ex-Googler
10 years, 5 months ago (2010-07-26 19:50:01 UTC) #1
Miranda Callahan
10 years, 5 months ago (2010-07-26 20:56:46 UTC) #2
On 2010/07/26 19:50:01, shess wrote:
> 

LGTM!

Powered by Google App Engine
This is Rietveld 408576698