Do all OOLing in the views code. linux_views now builds clean with the clang plugin.

views/controls/menu/menu_delegate.cc

Index: views/controls/menu/menu_delegate.cc
diff --git a/views/controls/menu/menu_delegate.cc b/views/controls/menu/menu_delegate.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e70fb171ad200f3abe00850e44b1a6c1eed859d1
--- /dev/null
+++ b/views/controls/menu/menu_delegate.cc
@@ -0,0 +1,113 @@
+// Copyright (c) 2011 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 "views/controls/menu/menu_delegate.h"
+
+namespace views {
+
+bool MenuDelegate::IsItemChecked(int id) const {
+  return false;
+}
+
+std::wstring MenuDelegate::GetLabel(int id) const {
+  return std::wstring();
+}
+
+std::wstring MenuDelegate::GetTooltipText(int id,
+                                          const gfx::Point& screen_loc) {
+  return std::wstring();
+}
+
+bool MenuDelegate::GetAccelerator(int id, Accelerator* accelerator) {
+  return false;
+}
+
+bool MenuDelegate::ShowContextMenu(MenuItemView* source,
+                                   int id,
+                                   const gfx::Point& p,
+                                   bool is_mouse_gesture) {
+  return false;
+}
+
+bool MenuDelegate::SupportsCommand(int id) const {
+  return true;
+}
+
+bool MenuDelegate::IsCommandEnabled(int id) const {
+  return true;
+}
+
+bool MenuDelegate::GetContextualLabel(int id, std::wstring* out) const {
+  return false;
+}
+
+bool MenuDelegate::ShouldCloseAllMenusOnExecute(int id) {
+  return true;
+}
+
+void MenuDelegate::ExecuteCommand(int id, int mouse_event_flags) {
+  ExecuteCommand(id);
+}
+
+bool MenuDelegate::IsTriggerableEvent(const MouseEvent& e) {
+  return e.IsLeftMouseButton() || e.IsRightMouseButton();
+}
+
+bool MenuDelegate::CanDrop(MenuItemView* menu, const OSExchangeData& data) {
+  return false;
+}
+
+bool MenuDelegate::GetDropFormats(
+    MenuItemView* menu,
+    int* formats,
+    std::set<OSExchangeData::CustomFormat>* custom_formats) {
+  return false;
+}
+
+bool MenuDelegate::AreDropTypesRequired(MenuItemView* menu) {
+  return false;
+}
+
+int MenuDelegate::GetDropOperation(MenuItemView* item,
+                                   const DropTargetEvent& event,
+                                   DropPosition* position) {
+  NOTREACHED() << "If you override CanDrop, you need to override this too";
+  return ui::DragDropTypes::DRAG_NONE;
+}
+
+int MenuDelegate::OnPerformDrop(MenuItemView* menu,
+                                DropPosition position,
+                                const DropTargetEvent& event) {
+  NOTREACHED() << "If you override CanDrop, you need to override this too";
+  return ui::DragDropTypes::DRAG_NONE;
+}
+
+bool MenuDelegate::CanDrag(MenuItemView* menu) {
+  return false;
+}
+
+void MenuDelegate::WriteDragData(MenuItemView* sender, OSExchangeData* data) {
+  NOTREACHED() << "If you override CanDrag, you must override this too.";
+}
+
+int MenuDelegate::GetDragOperations(MenuItemView* sender) {
+  NOTREACHED() << "If you override CanDrag, you must override this too.";
+  return 0;
+}
+
+MenuItemView* MenuDelegate::GetSiblingMenu(MenuItemView* menu,
+                                           const gfx::Point& screen_point,
+                                           MenuItemView::AnchorPosition* anchor,
+                                           bool* has_mnemonics,
+                                           MenuButton** button) {
+  return NULL;
+}
+
+int MenuDelegate::GetMaxWidthForMenu() {
+  // NOTE: this needs to be large enough to accommodate the wrench menu with
+  // big fonts.
+  return 800;
+}
+
+}  // namespace views