retry r112077 - recently closed menu: respect middle click

chrome/browser/resources/ntp4/apps_page.js

 // Copyright (c) 2011 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.
 
 cr.define('ntp4', function() {
   'use strict';
 
   var localStrings = new LocalStrings;
 
   var APP_LAUNCH = {
@@ skipping 428 matching lines @@
       var url = !this.appData_.is_webstore ? '' :
           is_promo ? appendParam(this.appsPromoLink_.href,
                                  'utm_source',
                                  'chrome-ntp-promo') :
                      appendParam(this.appData_.url,
                                  'utm_source',
                                  'chrome-ntp-icon');
 
       chrome.send('launchApp',
                   [this.appId, APP_LAUNCH.NTP_APPS_MAXIMIZED, url,
-                   e.altKey, e.ctrlKey, e.metaKey, e.shiftKey, e.button]);
+                   e.button, e.altKey, e.ctrlKey, e.metaKey, e.shiftKey]);
 
       // Don't allow the click to trigger a link or anything
       e.preventDefault();
     },
 
     /**
      * Invoked when the user presses a key while the app is focused.
      * @param {Event} e The key event.
      * @private
      */
     onKeydown_: function(e) {
       if (e.keyIdentifier == 'Enter') {
         chrome.send('launchApp',
                     [this.appId, APP_LAUNCH.NTP_APPS_MAXIMIZED, '',
-                     e.altKey, e.ctrlKey, e.metaKey, e.shiftKey, 0]);
+                     0, e.altKey, e.ctrlKey, e.metaKey, e.shiftKey]);
         e.preventDefault();
         e.stopPropagation();
       }
     },
 
     /**
      * Adds a node to the list of targets that will launch the app. This list
      * is also used in onMousedown to determine whether the app contents should
      * be shown as active (if we don't do this, then clicking anywhere in
      * appContents, even a part that is outside the ideally clickable region,
@@ skipping 326 matching lines @@
     appNotificationChanged: appNotificationChanged,
     AppsPage: AppsPage,
     launchAppAfterEnable: launchAppAfterEnable,
   };
 });
 
 // TODO(estade): update the content handlers to use ntp namespace instead of
 // making these global.
 var appNotificationChanged = ntp4.appNotificationChanged;
 var launchAppAfterEnable = ntp4.launchAppAfterEnable;