Context menus should appear above the touch point if invoked by long press.

chrome/browser/ui/views/tab_contents/render_view_context_menu_views.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 "chrome/browser/ui/views/tab_contents/render_view_context_menu_views.h"
 
 #include "base/logging.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "content/public/browser/render_widget_host_view.h"
@@ skipping 24 matching lines @@
 
 #if !defined(OS_WIN)
 // static
 RenderViewContextMenuViews* RenderViewContextMenuViews::Create(
     content::WebContents* tab_contents,
     const content::ContextMenuParams& params) {
   return new RenderViewContextMenuViews(tab_contents, params);
 }
 #endif  // OS_WIN
 
-void RenderViewContextMenuViews::RunMenuAt(views::Widget* parent,
-                                           const gfx::Point& point) {
+void RenderViewContextMenuViews::RunMenuAt(
+    views::Widget* parent,
+    const gfx::Point& point,
+    const content::ContextMenuSourceType& type) {
+  views::MenuItemView::AnchorPosition anchor_position =
+      type == content::CONTEXT_MENU_SOURCE_GESTURE ?
+          views::MenuItemView::BOTTOMCENTER : views::MenuItemView::TOPLEFT;

[sky, 2012/09/06 16:35:03]

Why do we want BOTTOMCENTER for long press?

[varunjain, 2012/09/06 16:43:45]

request from UX was to display the context menu centered above the finger for
long press. For right click with mouse, the context menu should remain where it
is.

   if (menu_runner_->RunMenuAt(parent, NULL, gfx::Rect(point, gfx::Size()),
-          views::MenuItemView::TOPLEFT, views::MenuRunner::HAS_MNEMONICS |
+      anchor_position, views::MenuRunner::HAS_MNEMONICS |
           views::MenuRunner::CONTEXT_MENU) ==
       views::MenuRunner::MENU_DELETED)
     return;
 }
 
 void RenderViewContextMenuViews::UpdateMenuItemStates() {
   menu_delegate_->BuildMenu(menu_);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 64 matching lines @@
   item->SetEnabled(enabled);
   item->SetTitle(title);
   item->SetVisible(!hidden);
 
   views::MenuItemView* parent = item->GetParentMenuItem();
   if (!parent)
     return;
 
   parent->ChildrenChanged();
 }