Move enum MenuAnchorPosition to reduce deps on menu_item_view.h

ui/base/ui_base_types.h

 // 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.
 
 #ifndef UI_BASE_UI_BASE_TYPES_H_
 #define UI_BASE_UI_BASE_TYPES_H_
 
 #include "ui/base/ui_base_export.h"
 
 namespace ui {
@@ skipping 35 matching lines @@
   MENU_SOURCE_NONE            = 0,
   MENU_SOURCE_MOUSE           = 1,
   MENU_SOURCE_KEYBOARD        = 2,
   MENU_SOURCE_TOUCH           = 3,
   MENU_SOURCE_TOUCH_EDIT_MENU = 4,
   MENU_SOURCE_TYPE_LAST = MENU_SOURCE_TOUCH_EDIT_MENU
 };
 
 UI_BASE_EXPORT MenuSourceType GetMenuSourceTypeForEvent(const ui::Event& event);
 
+// Where a popup menu should be anchored to for non-RTL languages. The opposite
+// position will be used if base::i18n:IsRTL() is true. The BUBBLE flags are
+// used when the menu should get enclosed by a bubble. Note that BUBBLE flags
+// should only be used with menus which have no children.
+enum MenuAnchorPosition {

[sky, 2014/04/29 15:30:57]

I'm fine with moving MenuAnchorPosition, but it's view specific. Maybe
ui/views/controls/menu/menu_types.h.

[tapted, 2014/04/30 08:32:56]

Done.

+  MENU_ANCHOR_TOPLEFT,
+  MENU_ANCHOR_TOPRIGHT,
+  MENU_ANCHOR_BOTTOMCENTER,
+  MENU_ANCHOR_BUBBLE_LEFT,
+  MENU_ANCHOR_BUBBLE_RIGHT,
+  MENU_ANCHOR_BUBBLE_ABOVE,
+  MENU_ANCHOR_BUBBLE_BELOW
+};
+
 }  // namespace ui
 
 #endif  // UI_BASE_UI_BASE_TYPES_H_

[tapted, 2014/04/29 11:53:14]

<link to interesting file>