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

Unified Diff: chrome/browser/cocoa/location_bar/autocomplete_text_field.mm

Issue 2805070: [Mac] First part of Omnibox decoration refactor. Enable ev bubble. (Closed) Base URL: git://codf21.jail/chromium.git
Patch Set: comment clarification Created 10 years, 5 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: chrome/browser/cocoa/location_bar/autocomplete_text_field.mm
diff --git a/chrome/browser/cocoa/location_bar/autocomplete_text_field.mm b/chrome/browser/cocoa/location_bar/autocomplete_text_field.mm
index bfc608091251382f6b303fabc62080638d0ab7d1..3a1dc322a99c96e613abd6493fbd0718718b728e 100644
--- a/chrome/browser/cocoa/location_bar/autocomplete_text_field.mm
+++ b/chrome/browser/cocoa/location_bar/autocomplete_text_field.mm
@@ -211,9 +211,9 @@
for (AutocompleteTextFieldIcon* icon in [cell layedOutIcons:fieldBounds])
[self addCursorRect:[icon rect] cursor:[NSCursor arrowCursor]];
- // Special-case the location image, since it is not in |-layedOutIcons|.
- const NSRect locationIconFrame = [cell locationIconFrameForFrame:fieldBounds];
- [self addCursorRect:locationIconFrame cursor:[NSCursor arrowCursor]];
+ // TODO(shess): This needs to traverse the LocationBarDecorations
+ // and put up a cursor for them, too. Except for the keyword-search
+ // stuff? Sigh.
}
// TODO(shess): -resetFieldEditorFrameIfNeeded is the place where
@@ -405,8 +405,4 @@
actionMenuForEvent:event inRect:[self bounds] ofView:self];
}
-- (NSPasteboard*)locationDragPasteboard {
- return [[self autocompleteTextFieldCell] locationDragPasteboard];
-}
-
@end

Powered by Google App Engine
This is Rietveld 408576698