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

Issue 2854051: [Mac] Convert omnibox keyword hint to decoration. (Closed)

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

Description

[Mac] Convert omnibox keyword hint to decoration. Additionally, convert to use more basic drawing rather than NSAttributedString, to save needing the text-attachment class. Strip out all of the unit-test code relating to decoration exclusion, as those cases are no longer possible in those ways. BUG=none TEST=Browse to www.google.com. Command-l, type google and should see "Press [tab] to search Google". TEST=Should have I-beam cursor over the "Press ..." phrase. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52928

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+320 lines, -507 lines) Patch
M chrome/browser/cocoa/location_bar/autocomplete_text_field_cell.h View 3 chunks +0 lines, -27 lines 0 comments Download
M chrome/browser/cocoa/location_bar/autocomplete_text_field_cell.mm View 10 chunks +3 lines, -201 lines 0 comments Download
M chrome/browser/cocoa/location_bar/autocomplete_text_field_cell_unittest.mm View 8 chunks +14 lines, -143 lines 0 comments Download
M chrome/browser/cocoa/location_bar/autocomplete_text_field_unittest.mm View 9 chunks +52 lines, -87 lines 0 comments Download
A chrome/browser/cocoa/location_bar/keyword_hint_decoration.h View 1 chunk +46 lines, -0 lines 0 comments Download
A chrome/browser/cocoa/location_bar/keyword_hint_decoration.mm View 1 chunk +133 lines, -0 lines 2 comments Download
A chrome/browser/cocoa/location_bar/keyword_hint_decoration_unittest.mm View 1 chunk +57 lines, -0 lines 0 comments Download
M chrome/browser/cocoa/location_bar/location_bar_view_mac.h View 3 chunks +5 lines, -4 lines 0 comments Download
M chrome/browser/cocoa/location_bar/location_bar_view_mac.mm View 6 chunks +7 lines, -45 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Scott Hess - ex-Googler
Last bit, now everything is a decoration.
10 years, 5 months ago (2010-07-17 06:01:30 UTC) #1
Scott Hess - ex-Googler
On 2010/07/17 06:01:30, shess wrote: > Last bit, now everything is a decoration. And sorry ...
10 years, 5 months ago (2010-07-17 06:03:00 UTC) #2
rohitrao (ping after 24h)
LGTM. Nice to see all that code go. It occurs to me that AutocompleteTextFieldCell no ...
10 years, 5 months ago (2010-07-19 16:57:33 UTC) #3
Scott Hess - ex-Googler
10 years, 5 months ago (2010-07-19 17:52:12 UTC) #4
On 2010/07/19 16:57:33, rohitrao wrote:
> LGTM.  Nice to see all that code go.
> 
> It occurs to me that AutocompleteTextFieldCell no longer really does anything
> autocomplete-specific :)

Just need to find somewhere else to use all of this black magic.

[Maybe findbar, but at some point I'm not sure whether the similarities _really_
outweigh the differences.  It wouldn't surprise me much to determine that we
could fork the two uses again and reduce the overall amount of code :-).]

http://codereview.chromium.org/2854051/diff/1/7
File chrome/browser/cocoa/location_bar/keyword_hint_decoration.mm (right):

http://codereview.chromium.org/2854051/diff/1/7#newcode34
chrome/browser/cocoa/location_bar/keyword_hint_decoration.mm:34: NSDictionary* d
=
On 2010/07/19 16:57:33, rohitrao wrote:
> Call this "attributes" instead of "d."

Done.

Powered by Google App Engine
This is Rietveld 408576698