Bug 946862: Disable vertical overscroll until bug 939480 is fixed. r=felipe
authorStephen Pohl <spohl.mozilla.bugs@gmail.com>
Thu, 05 Dec 2013 15:38:32 -0500
changeset 173766 1c645ff95f2c9db5339bcf5bf8df5f914edf8a5a
parent 173765 aebd77677855992dc918b4ca302ac070ee83de35
child 173767 015be0f83e7a38d3f92081fe3c42daeec4ad3155
push id3224
push userlsblakk@mozilla.com
push dateTue, 04 Feb 2014 01:06:49 +0000
treeherdermozilla-beta@60c04d0987f1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfelipe
bugs946862, 939480
milestone28.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 946862: Disable vertical overscroll until bug 939480 is fixed. r=felipe
browser/base/content/browser-gestureSupport.js
--- a/browser/base/content/browser-gestureSupport.js
+++ b/browser/base/content/browser-gestureSupport.js
@@ -198,20 +198,19 @@ let gGestureSupport = {
       }
       isVerticalSwipe = true;
     } else if (aEvent.direction == aEvent.DIRECTION_DOWN) {
       if (content.pageYOffset < content.scrollMaxY) {
         return false;
       }
       isVerticalSwipe = true;
     }
-    if (isVerticalSwipe && !gHistorySwipeAnimation.active) {
-      // Unlike horizontal swipes (which can navigate history even when
-      // swipe animations are turned off) vertical swipes should not be tracked
-      // if animations (bounce effect) aren't enabled.
+    if (isVerticalSwipe) {
+      // Vertical overscroll has been temporarily disabled until bug 939480 is
+      // fixed.
       return false;
     }
 
     let canGoBack = gHistorySwipeAnimation.canGoBack();
     let canGoForward = gHistorySwipeAnimation.canGoForward();
     let isLTR = gHistorySwipeAnimation.isLTR;
 
     if (canGoBack) {