Remove WebContentsDelegate::ExecuteContextMenuCommand since it's not called by content. Since it's …

chrome/browser/ui/views/external_tab_container_win.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/ui/views/external_tab_container_win.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 313 matching lines @@
 }
 
 void ExternalTabContainerWin::SetTabHandle(int handle) {
   tab_handle_ = handle;
 }
 
 int ExternalTabContainerWin::GetTabHandle() const {
   return tab_handle_;
 }
 
+bool ExternalTabContainerWin::ExecuteContextMenuCommand(int command) {
+  if (!external_context_menu_.get()) {
+    NOTREACHED();
+    return false;
+  }
+
+  switch (command) {
+    case IDS_CONTENT_CONTEXT_SAVEAUDIOAS:
+    case IDS_CONTENT_CONTEXT_SAVEVIDEOAS:
+    case IDS_CONTENT_CONTEXT_SAVEIMAGEAS:
+    case IDS_CONTENT_CONTEXT_SAVELINKAS: {
+      NOTREACHED();  // Should be handled in host.
+      break;
+    }
+  }
+
+  external_context_menu_->ExecuteCommand(command);
+  return true;
+}
+
 void ExternalTabContainerWin::RunUnloadHandlers(IPC::Message* reply_message) {
   if (!automation_) {
     delete reply_message;
     return;
   }
 
   // If we have a pending unload message, then just respond back to this
   // request and continue processing the previous unload message.
   if (unload_reply_message_) {
      AutomationMsg_RunUnloadHandlers::WriteReplyParams(reply_message, true);
@@ skipping 244 matching lines @@
   if (automation_) {
     string16 url_string = CA2W(url.spec().c_str());
     automation_->Send(
         new AutomationMsg_UpdateTargetUrl(tab_handle_, url_string));
   }
 }
 
 void ExternalTabContainerWin::ContentsZoomChange(bool zoom_in) {
 }
 
-gfx::NativeWindow ExternalTabContainerWin::GetFrameNativeWindow() {
-  return GetNativeView();
-}
-
 bool ExternalTabContainerWin::TakeFocus(content::WebContents* source,
                                         bool reverse) {
   if (automation_) {
     automation_->Send(new AutomationMsg_TabbedOut(tab_handle_,
         base::win::IsShiftPressed()));
   }
 
   return true;
 }
 
@@ skipping 92 matching lines @@
 
   bool rtl = base::i18n::IsRTL();
   automation_->Send(
       new AutomationMsg_ForwardContextMenuToExternalHost(tab_handle_,
           *context_menu_model,
           rtl ? TPM_RIGHTALIGN : TPM_LEFTALIGN, ipc_params));
 
   return true;
 }
 
-bool ExternalTabContainerWin::ExecuteContextMenuCommand(int command) {
-  if (!external_context_menu_.get()) {
-    NOTREACHED();
-    return false;
-  }
-
-  switch (command) {
-    case IDS_CONTENT_CONTEXT_SAVEAUDIOAS:
-    case IDS_CONTENT_CONTEXT_SAVEVIDEOAS:
-    case IDS_CONTENT_CONTEXT_SAVEIMAGEAS:
-    case IDS_CONTENT_CONTEXT_SAVELINKAS: {
-      NOTREACHED();  // Should be handled in host.
-      break;
-    }
-  }
-
-  external_context_menu_->ExecuteCommand(command);
-  return true;
-}
-
 bool ExternalTabContainerWin::PreHandleKeyboardEvent(
     content::WebContents* source,
     const NativeWebKeyboardEvent& event,
     bool* is_keyboard_shortcut) {
   return false;
 }
 
 void ExternalTabContainerWin::HandleKeyboardEvent(
     content::WebContents* source,
     const NativeWebKeyboardEvent& event) {
@@ skipping 529 matching lines @@
   if (params.disposition == CURRENT_TAB) {
     DCHECK(route_all_top_level_navigations_);
     forward_params.disposition = NEW_FOREGROUND_TAB;
   }
   WebContents* new_contents =
       ExternalTabContainerWin::OpenURLFromTab(source, forward_params);
   // support only one navigation for a dummy tab before it is killed.
   ::DestroyWindow(GetNativeView());
   return new_contents;
 }