Fixes regression in menus. Specifically selecting by way of press,...

chrome/views/menu.cc

 // Copyright (c) 2006-2008 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/views/menu.h"
 
 #include <atlbase.h>
 #include <atlcrack.h>
 #include <atlapp.h>
 #include <atlframe.h>
@@ skipping 536 matching lines @@
 
   for (size_t i = 0; i < submenus_.size(); ++i)
     submenus_[i]->SetMenuInfo();
 }
 
 void Menu::RunMenuAt(int x, int y) {
   SetMenuInfo();
 
   delegate_->MenuWillShow();
 
-  // Show the menu.  Blocks until the menu is dismissed or an item is chosen.
+  // NOTE: we don't use TPM_RIGHTBUTTON here as it breaks selecting by way of
+  // press, drag, release. See bugs 718 and 8560.
   UINT flags =
-      GetTPMAlignFlags() | TPM_RIGHTBUTTON | TPM_RETURNCMD | TPM_RECURSE;
+      GetTPMAlignFlags() | TPM_LEFTBUTTON | TPM_RETURNCMD | TPM_RECURSE;
   is_menu_visible_ = true;
   DCHECK(owner_);
   // In order for context menus on menus to work, the context menu needs to
   // share the same window as the first menu is parented to.
   bool created_host = false;
   if (!active_host_window) {
     created_host = true;
     active_host_window = new MenuHostWindow(this, owner_);
   }
   UINT selected_id =
@@ skipping 11 matching lines @@
 
 void Menu::Cancel() {
   DCHECK(is_menu_visible_);
   EndMenu();
 }
 
 int Menu::ItemCount() {
   return GetMenuItemCount(menu_);
 }