Eliminate faux-RTTI code from BrowserView.

chrome/browser/ui/views/frame/browser_view.cc

 // Copyright 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/frame/browser_view.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 1443 matching lines @@
                                                         GetFocusManager());
 }
 
 // TODO(devint): http://b/issue?id=1117225 Cut, Copy, and Paste are always
 // enabled in the page menu regardless of whether the command will do
 // anything. When someone selects the menu item, we just act as if they hit
 // the keyboard shortcut for the command by sending the associated key press
 // to windows. The real fix to this bug is to disable the commands when they
 // won't do anything. We'll need something like an overall clipboard command
 // manager to do that.
-void BrowserView::Cut() {
-  // If a WebContent is focused, call WebContents::Cut. Otherwise, e.g. if
-  // Omnibox is focused, send a Ctrl+x key event to Chrome. Using RWH interface
-  // rather than the fake key event for a WebContent is important since the fake
-  // event might be consumed by the web content (crbug.com/137908).
-  DoCutCopyPaste(&content::WebContents::Cut, IDS_APP_CUT);
-}
+void BrowserView::CutCopyPaste(int command_id) {
+  // If a WebContents is focused, call its member method.
+  WebContents* contents = browser_->tab_strip_model()->GetActiveWebContents();
+  if (contents) {
+    void (WebContents::*method)();
+    if (command_id == IDC_CUT)
+      method = &content::WebContents::Cut;
+    else if (command_id == IDC_COPY)
+      method = &content::WebContents::Copy;
+    else
+      method = &content::WebContents::Paste;
+    if (DoCutCopyPasteForWebContents(contents, method))
+      return;
 
-void BrowserView::Copy() {
-  DoCutCopyPaste(&content::WebContents::Copy, IDS_APP_COPY);
-}
+    WebContents* devtools =
+        DevToolsWindow::GetInTabWebContents(contents, nullptr);
+    if (devtools && DoCutCopyPasteForWebContents(devtools, method))
+      return;
+  }
 
-void BrowserView::Paste() {
-  DoCutCopyPaste(&content::WebContents::Paste, IDS_APP_PASTE);
+  // Execute the command on the focused view, if possible, by calling
+  // AcceleratorPressed().  Textfields override this to translate and execute
+  // the provided accelerator.
+  views::View* focused = GetFocusManager()->GetFocusedView();
+  if (focused->CanHandleAccelerators()) {
+    ui::Accelerator accelerator;
+    GetAccelerator(command_id, &accelerator);
+    focused->AcceleratorPressed(accelerator);
+  }
 }
 
 WindowOpenDisposition BrowserView::GetDispositionForPopupBounds(
     const gfx::Rect& bounds) {
   return NEW_POPUP;
 }
 
 FindBar* BrowserView::CreateFindBar() {
   return chrome::CreateFindBar(this);
 }
@@ skipping 1019 matching lines @@
 void BrowserView::ExecuteExtensionCommand(
     const extensions::Extension* extension,
     const extensions::Command& command) {
   toolbar_->ExecuteExtensionCommand(extension, command);
 }
 
 ExclusiveAccessContext* BrowserView::GetExclusiveAccessContext() {
   return this;
 }
 
-void BrowserView::DoCutCopyPaste(void (WebContents::*method)(),
-                                 int command_id) {
-  WebContents* contents = browser_->tab_strip_model()->GetActiveWebContents();
-  if (!contents)
-    return;
-  if (DoCutCopyPasteForWebContents(contents, method))
-    return;
-
-  WebContents* devtools = DevToolsWindow::GetInTabWebContents(contents,
-                                                              nullptr);
-  if (devtools && DoCutCopyPasteForWebContents(devtools, method))
-    return;
-
-  views::FocusManager* focus_manager = GetFocusManager();
-  views::View* focused = focus_manager->GetFocusedView();
-  if (focused &&
-      (!strcmp(focused->GetClassName(), views::Textfield::kViewClassName) ||
-       !strcmp(focused->GetClassName(), OmniboxViewViews::kViewClassName))) {
-    views::Textfield* textfield = static_cast<views::Textfield*>(focused);
-    textfield->ExecuteCommand(command_id);
-  }
-}
-
 bool BrowserView::DoCutCopyPasteForWebContents(
     WebContents* contents,
     void (WebContents::*method)()) {
+  // Using RWH interface rather than a fake key event for a WebContent is
+  // important since a fake event might be consumed by the web content.
   if (contents->GetRenderWidgetHostView()->HasFocus()) {
     (contents->*method)();
     return true;
   }
 
   return false;
 }
 
 void BrowserView::ActivateAppModalDialog() const {
   // If another browser is app modal, flash and activate the modal browser.
@@ skipping 59 matching lines @@
   return immersive_mode_controller()->IsEnabled();
 }
 
 views::Widget* BrowserView::GetBubbleAssociatedWidget() {
   return GetWidget();
 }
 
 gfx::Rect BrowserView::GetTopContainerBoundsInScreen() {
   return top_container_->GetBoundsInScreen();
 }