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

chrome/browser/resources/ntp4/page_switcher.js

Index: chrome/browser/resources/ntp4/page_switcher.js
diff --git a/chrome/browser/resources/ntp4/page_switcher.js b/chrome/browser/resources/ntp4/page_switcher.js
index 00f51f9c49c8bcaccf8b0bc30b49a5079bdf1d17..2ae7f758aa1bc445ebcdd05e486b3bf0456ca94f 100644
--- a/chrome/browser/resources/ntp4/page_switcher.js
+++ b/chrome/browser/resources/ntp4/page_switcher.js
@@ -19,7 +19,6 @@ cr.define('ntp', function() {
       el.__proto__ = PageSwitcher.template;
 
       el.addEventListener('click', el.activate_);
-      el.addEventListener('mousewheel', el.onMouseWheel_);
 
       el.direction_ = el.id == 'page-switcher-start' ? -1 : 1;
 
@@ -37,18 +36,6 @@ cr.define('ntp', function() {
       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;