Add ContextMenuSourceType to views::ContextMenuController::ShowContextMenuForView.

ui/views/controls/menu/menu_runner.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_VIEWS_CONTROLS_MENU_MENU_RUNNER_H_
 #define UI_VIEWS_CONTROLS_MENU_MENU_RUNNER_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 79 matching lines @@
   // Typically callers should NOT do any processing if this returns
   // MENU_DELETED.
   // If |anchor| uses a |BUBBLE_..| type, the bounds will get determined by
   // using |bounds| as the thing to point at in screen coordinates.
   RunResult RunMenuAt(Widget* parent,
                       MenuButton* button,
                       const gfx::Rect& bounds,
                       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;
 
   // Hides and cancels the menu. This does nothing if the menu is not open.
   void Cancel();
 
   // Returns the time from the event which closed the menu - or 0.
   base::TimeDelta closing_event_time() const;
 
  private:
@@ skipping 22 matching lines @@
 
  protected:
   DisplayChangeListener() {}
 };
 
 }
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_MENU_MENU_RUNNER_H_