Remove search::GetSearchTerms since it always returns empty string

chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm

Index: chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
diff --git a/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm b/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
index 75a778970012546e8a38d2a1c469b5dfed94a768..b9aba854d40f9d43a3d79969b9d6bc4fc4383a19 100644
--- a/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
+++ b/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
@@ -935,11 +935,6 @@ void OmniboxViewMac::OnMouseDown(NSInteger button_number) {
     model()->SetCaretVisibility(true);
 }
 
-bool OmniboxViewMac::ShouldSelectAllOnMouseDown() {
-  return !controller()->GetToolbarModel()->WouldPerformSearchTermReplacement(
-      false);
-}
-
 bool OmniboxViewMac::CanCopy() {
   const NSRange selection = GetSelectedRange();
   return selection.length > 0;
@@ -952,15 +947,11 @@ base::scoped_nsobject<NSPasteboardItem> OmniboxViewMac::CreatePasteboardItem() {
   base::string16 text = base::SysNSStringToUTF16(
       [[field_ stringValue] substringWithRange:selection]);
 
-  // Copy the URL unless this is the search URL and it's being replaced by the
-  // Extended Instant API.
+  // Copy the URL.
   GURL url;
   bool write_url = false;
-  if (!controller()->GetToolbarModel()->WouldPerformSearchTermReplacement(
-      false)) {
-    model()->AdjustTextForCopy(selection.location, IsSelectAll(), &text, &url,
-                               &write_url);
-  }
+  model()->AdjustTextForCopy(selection.location, IsSelectAll(), &text, &url,
+                             &write_url);
 
   if (IsSelectAll())
     UMA_HISTOGRAM_COUNTS(OmniboxEditModel::kCutOrCopyAllTextHistogram, 1);
@@ -980,11 +971,6 @@ void OmniboxViewMac::CopyToPasteboard(NSPasteboard* pboard) {
   [pboard writeObjects:@[ item.get() ]];
 }
 
-void OmniboxViewMac::ShowURL() {
-  DCHECK(ShouldEnableShowURL());
-  OmniboxView::ShowURL();

[Peter Kasting, 2016/08/13 05:03:44]

Remember to eventually rip out the other call chain that leads to this function
(which I think goes through LocationBarView::ShowURL()), and then rip out the
function itself.

[Marc Treib, 2016/08/16 12:00:17]

Yup, follow-up on the way!

-}
-
 void OmniboxViewMac::OnPaste() {
   // This code currently expects |field_| to be focused.
   DCHECK([field_ currentEditor]);
@@ -1017,14 +1003,6 @@ void OmniboxViewMac::OnPaste() {
   }
 }
 
-// TODO(dominich): Move to OmniboxView base class? Currently this is defined on
-// the AutocompleteTextFieldObserver but the logic is shared between all
-// platforms. Some refactor might be necessary to simplify this. Or at least
-// this method could call the OmniboxView version.
-bool OmniboxViewMac::ShouldEnableShowURL() {
-  return controller()->GetToolbarModel()->WouldReplaceURL();
-}
-
 bool OmniboxViewMac::CanPasteAndGo() {
   return model()->CanPasteAndGo(GetClipboardText());
 }