Context menus on view elements should be positioned above the finger when

ui/views/controls/menu/menu_runner.cc

 // 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.
 
 #include "ui/views/controls/menu/menu_runner.h"
 
 #include <set>
 
 #include "ui/views/controls/button/menu_button.h"
 #include "ui/views/controls/menu/menu_controller.h"
 #include "ui/views/controls/menu/menu_controller_delegate.h"
 #include "ui/views/controls/menu/menu_delegate.h"
+#include "ui/views/widget/widget.h"
 
 #if defined(OS_WIN)
 #include "base/win/win_util.h"
 #endif
 
+namespace {
+// If a context menu is invoked by touch, we shift the menu by this offset so
+// that the finger does not obscure the menu.
+const int kCenteredContextMenuYOffset = -15;
+}  // namespace
+
 namespace views {
 
 namespace internal {
 
 // Manages the menu. To destroy a MenuRunnerImpl invoke Release(). Release()
 // deletes immediately if the menu isn't showing. If the menu is showing
 // Release() cancels the menu and when the nested RunMenuAt() call returns
 // deletes itself and the menu.
 class MenuRunnerImpl : public internal::MenuControllerDelegate {
  public:
@@ skipping 249 matching lines @@
                                             const gfx::Rect& bounds,
                                             MenuItemView::AnchorPosition anchor,
                                             int32 types) {
   // The parent of the nested menu will have created a DisplayChangeListener, so
   // we avoid creating a DisplayChangeListener if nested. Drop menus are
   // transient, so we don't cancel in that case.
   if ((types & (IS_NESTED | FOR_DROP)) == 0) {
     display_change_listener_.reset(
         internal::DisplayChangeListener::Create(parent, this));
   }
-  return holder_->RunMenuAt(parent, button, bounds, anchor, types);
+  if ((types & MenuRunner::CONTEXT_MENU) && parent->GetCurrentEvent())
+    anchor = parent->GetCurrentEvent()->IsGestureEvent() ?
+        MenuItemView::BOTTOMCENTER : MenuItemView::TOPLEFT;
+  gfx::Rect menu_bounds = bounds;
+  if (anchor == MenuItemView::BOTTOMCENTER)

[sky, 2012/09/14 17:19:57]

Can this be moved to UpdateInitialLocation?

[varunjain, 2012/09/14 18:46:06]

Done.

+    menu_bounds.Offset(0, kCenteredContextMenuYOffset);
+
+  return holder_->RunMenuAt(parent, button, menu_bounds, anchor, types);
 }
 
 bool MenuRunner::IsRunning() const {
   return holder_->running();
 }
 
 void MenuRunner::Cancel() {
   holder_->Cancel();
 }
 
 }  // namespace views