Add ContextMenuSourceType to views::ContextMenuController::ShowContextMenuForView.

ui/views/controls/menu/menu_runner.h

Index: ui/views/controls/menu/menu_runner.h
diff --git a/ui/views/controls/menu/menu_runner.h b/ui/views/controls/menu/menu_runner.h
index 2e9a133ad1348f8f448d9d2d4183b89455971db9..acfaa32ce9a43f029b05f13db6a3f930e0466b03 100644
--- a/ui/views/controls/menu/menu_runner.h
+++ b/ui/views/controls/menu/menu_runner.h
@@ -97,6 +97,12 @@ class VIEWS_EXPORT MenuRunner {
                       MenuItemView::AnchorPosition anchor,
                       int32 types) WARN_UNUSED_RESULT;
 
+  RunResult RunContextMenuAt(Widget* parent,

[sky, 2013/06/13 19:54:53]

We already have RunTypes::CONTEXT_MENU, so we don't need another method for
context menus. How about adding MenuSourceType as a parameter to RunMenuAt?
MenuSourceType could either be defined here, rename ContextMenuSourceType to
MenuSourceType (I haven't investigated the later to make sure that makes sense.

[varunjain, 2013/06/13 20:26:43]

The requirement here is to identify, 1) if its a context menu or not, 2) if its
a context menu, whats the menu source.
How does this look as the definition of MenuSourceType:
MenuSourceType {
  MENU_SOURCE_TYPE_NONE, // non-context menu source type
  MENU_SOURCE_TYPE_CONTEXT_MOUSE,
  MENU_SOURCE_TYPE_CONTEXT_TOUCH,
  MENU_SOURCE_TYPE_CONTEXT_KEYBOARD,
  MENU_SOURCE_TYPE_CONTEXT_TOUCH_EDITING,
}
Also, what are your thoughts on sadrul's suggestion of combining various
*SourceType enums? Having a source type enum specific to menus will make that
harder.

+                             MenuButton* button,
+                             const gfx::Rect& bounds,
+                             ui::ContextMenuSourceType source,
+                             int32 types) WARN_UNUSED_RESULT;
+
   // Returns true if we're in a nested message loop running the menu.
   bool IsRunning() const;