Mac: Prevent NTP mousewheel events from being suppressed on Lion.

chrome/browser/resources/ntp4/page_switcher.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.
 
 /**
  * @fileoverview Page switcher
  * This is the class for the left and right navigation arrows that switch
  * between pages.
  */
 cr.define('ntp', function() {
 
   function PageSwitcher() {
   }
 
   PageSwitcher.template = {
     __proto__: HTMLButtonElement.prototype,
 
     decorate: function(el) {
       el.__proto__ = PageSwitcher.template;
 
       el.addEventListener('click', el.activate_);
-      el.addEventListener('mousewheel', el.onMouseWheel_);
 
       el.direction_ = el.id == 'page-switcher-start' ? -1 : 1;
 
       el.dragWrapper_ = new cr.ui.DragWrapper(el, el);
     },
 
     /**
      * Activate the switcher (go to the next card).
      * @private
      */
     activate_: function() {
       var cardSlider = ntp.getCardSlider();
       var index = cardSlider.currentCard + this.direction_;
       var numCards = cardSlider.cardCount - 1;
       cardSlider.selectCard(Math.max(0, Math.min(index, numCards)), true);
     },
 
-    /**
-     * Handler for the mousewheel event on a pager. We pass through the scroll
-     * to the page. This is necssary because the page is our sibling in the DOM
-     * hierarchy, so the event won't naturally pass through to it.
-     * @param {Event} e The mousewheel event.
-     * @private
-     */
-    onMouseWheel_: function(e) {
-      var page = ntp.getCardSlider().currentCardValue;
-      page.handleMouseWheel(e);

[Patrick Dubroy, 2012/04/17 17:28:23]

This code was actually not necessary at all, because the page switcher is a
descendant of the card slider frame, which handles mousewheel.

-    },
-
     shouldAcceptDrag: function(e) {
       // We allow all drags to trigger the page switching effect.
       return true;
     },
 
     doDragEnter: function(e) {
       this.scheduleDelayedSwitch_();
       this.doDragOver(e);
     },
 
@@ skipping 49 matching lines @@
         this.dragNavTimeout_ = null;
       }
     },
 
   };
 
   return {
     initializePageSwitcher: PageSwitcher.template.decorate
   };
 });