Ignore mouseup shortly after showing cr.ui.Menu

ui/webui/resources/js/cr/ui/context_menu_handler.js

 // 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.
 
 // require: event_target.js
 
 cr.define('cr.ui', function() {
   /** @const */ var EventTarget = cr.EventTarget;
   /** @const */ var Menu = cr.ui.Menu;
 
@@ skipping 25 matching lines @@
      *     event).
      * @param {!cr.ui.Menu} menu The menu to show.
      */
     showMenu: function(e, menu) {
       menu.updateCommands(assertInstanceof(e.currentTarget, Node));
       if (!menu.hasVisibleItems())
         return;
 
       this.menu_ = menu;
       menu.classList.remove('hide-delayed');
-      menu.hidden = false;
+      menu.show();
       menu.contextElement = e.currentTarget;
 
       // When the menu is shown we steal a lot of events.
       var doc = menu.ownerDocument;
       var win = doc.defaultView;
       this.showingEvents_.add(doc, 'keydown', this, true);
       this.showingEvents_.add(doc, 'mousedown', this, true);
       this.showingEvents_.add(doc, 'touchstart', this, true);
       this.showingEvents_.add(doc, 'focus', this);
       this.showingEvents_.add(win, 'popstate', this);
@@ skipping 16 matching lines @@
      */
     hideMenu: function(opt_hideType) {
       var menu = this.menu;
       if (!menu)
         return;
 
       if (opt_hideType == cr.ui.HideType.DELAYED)
         menu.classList.add('hide-delayed');
       else
         menu.classList.remove('hide-delayed');
-      menu.hidden = true;
+      menu.hide();
       var originalContextElement = menu.contextElement;
       menu.contextElement = null;
       this.showingEvents_.removeAll();
       menu.selectedIndex = -1;
       this.menu_ = null;
 
       // On windows we might hide the menu in a right mouse button up and if
       // that is the case we wait some short period before we allow the menu
       // to be shown again.
       this.hideTimestamp_ = cr.isWindows ? Date.now() : 0;
@@ skipping 186 matching lines @@
    * The singleton context menu handler.
    * @type {!ContextMenuHandler}
    */
   var contextMenuHandler = new ContextMenuHandler;
 
   // Export
   return {
     contextMenuHandler: contextMenuHandler,
   };
 });