[Mac] Convert Omnibox paste-and-go to use AutocompleteTextFieldObserver.

chrome/browser/cocoa/autocomplete_text_field_editor.mm

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/cocoa/autocomplete_text_field_editor.h"
 
 #include "app/l10n_util_mac.h"
 #include "base/string_util.h"
 #include "grit/generated_resources.h"
 #include "base/sys_string_conversions.h"
@@ skipping 38 matching lines @@
 
 - (void)paste:(id)sender {
   AutocompleteTextFieldObserver* observer = [self observer];
   DCHECK(observer);
   if (observer) {
     observer->OnPaste();
   }
 }
 
 - (void)pasteAndGo:sender {
-  id delegate = [self delegate];
-  if ([delegate respondsToSelector:@selector(textDidPasteAndGo:)])
-    [delegate textDidPasteAndGo:self];
+  AutocompleteTextFieldObserver* observer = [self observer];
+  DCHECK(observer);
+  if (observer) {
+    observer->OnPasteAndGo();
+  }
 }
 
 // We have rich text, but it shouldn't be modified by the user, so
 // don't update the font panel.  In theory, -setUsesFontPanel: should
 // accomplish this, but that gets called frequently with YES when
 // NSTextField and NSTextView synchronize their contents.  That is
 // probably unavoidable because in most cases having rich text in the
 // field you probably would expect it to update the font panel.
 - (void)updateFontPanel {
 }
 
 // No ruler bar, so don't update any of that state, either.
 - (void)updateRuler {
 }
 
 - (NSMenu*)menuForEvent:(NSEvent*)event {
   NSMenu* menu = [[[NSMenu alloc] initWithTitle:@"TITLE"] autorelease];
-  [menu insertItemWithTitle:l10n_util::GetNSStringWithFixup(IDS_CUT)
-        action:@selector(cut:)
-        keyEquivalent:@"" atIndex:0];
-  [menu insertItemWithTitle:l10n_util::GetNSStringWithFixup(IDS_COPY)
-        action:@selector(copy:)
-        keyEquivalent:@"" atIndex:1];
-  [menu insertItemWithTitle:l10n_util::GetNSStringWithFixup(IDS_PASTE)
-        action:@selector(paste:)
-        keyEquivalent:@"" atIndex:2];
+  [menu addItemWithTitle:l10n_util::GetNSStringWithFixup(IDS_CUT)
+                  action:@selector(cut:)
+           keyEquivalent:@""];
+  [menu addItemWithTitle:l10n_util::GetNSStringWithFixup(IDS_COPY)
+                  action:@selector(copy:)
+           keyEquivalent:@""];
+  [menu addItemWithTitle:l10n_util::GetNSStringWithFixup(IDS_PASTE)
+                  action:@selector(paste:)
+           keyEquivalent:@""];
 
   // Paste and go/search.
-  id delegate = [self delegate];
+  AutocompleteTextFieldObserver* observer = [self observer];
+  DCHECK(observer);
+  if (observer && observer->CanPasteAndGo()) {
+    const int string_id = observer->GetPasteActionStringId();
+    NSString* label = l10n_util::GetNSStringWithFixup(string_id);
 
-  if ([delegate respondsToSelector:@selector(textPasteActionString:)]) {
-    NSString* label = [delegate textPasteActionString:self];
-    // TODO(rohitrao): If the clipboard is empty, should we show a greyed-out
-    // "Paste and Go" or nothing at all?
+    // TODO(rohitrao): If the clipboard is empty, should we show a
+    // greyed-out "Paste and Go" or nothing at all?
     if (label) {
-      [menu insertItemWithTitle:label action:@selector(pasteAndGo:)
-            keyEquivalent:@"" atIndex:3];
+      [menu addItemWithTitle:label
+                      action:@selector(pasteAndGo:)
+               keyEquivalent:@""];
     }
   }
 
   return menu;
 }
 
 @end