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 174740 1c645ff95f2c9db5339bcf5bf8df5f914edf8a5a
parent 174739 aebd77677855992dc918b4ca302ac070ee83de35
child 174741 015be0f83e7a38d3f92081fe3c42daeec4ad3155
push id445
push userffxbld
push dateMon, 10 Mar 2014 22:05:19 +0000
treeherdermozilla-release@dc38b741b04e [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) {