Closed Bug 1252708 Opened 8 years ago Closed 8 years ago

Able to scroll past the end of the page on American Airlines [aa.com]

Categories

(Firefox for Android Graveyard :: Toolbar, defect)

47 Branch
All
Android
defect
Not set
normal

Tracking

(firefox46 unaffected, firefox47 unaffected, firefox48 fixed)

RESOLVED FIXED
Firefox 48
Tracking Status
firefox46 --- unaffected
firefox47 --- unaffected
firefox48 --- fixed

People

(Reporter: kbrosnan, Assigned: rbarker)

References

Details

Attachments

(1 file)

1) Visit http://www.aa.com
2) perform a slow fling (leave your finger on the device to best see the issue)
3) notice the white bar at the bottom of the screen
4) content snaps back to the bottom of the screen on finger release

Using a Nexus 5 running Android 6.0
This is a bug from the dynamic toolbar implementation, I think. Should happen on any page that is long enough to scroll off the toolbar and short enough to reach the bottom of the page in a single swipe. The resize doesn't happen until the finger is lifted and until that happens the scroll offset is allowed to go to the higher value (based on the smaller window size). We should be able to fix this either in Java or in some Fennec-specific compositor code like what we have already for fixed-pos items.
Assignee: nobody → rbarker
Comment on attachment 8730977 [details] [diff] [review]
0001-Bug-1252708-Able-to-scroll-past-the-end-of-the-page-in-Fennec-with-APZ-enabled-r-16031516-980f413.patch

Review of attachment 8730977 [details] [diff] [review]:
-----------------------------------------------------------------

This is a little scary to me and it would be good to test panning madly up and down (with the finger held down) on some heavy pages with resize event listeners. Also run through the tests on http://people.mozilla.org/~kgupta/bug/1180295.html (ignore step 11, since the zoomed view is disabled) to make sure there's no expected behaviour. If all that is good then I don't have any objections.

::: mobile/android/base/java/org/mozilla/gecko/gfx/DynamicToolbarAnimator.java
@@ +440,5 @@
>              return false;
>          }
>  
> +        if (mToolbarTranslation == mMaxTranslation) {
> +            shiftLayerView(mMaxTranslation);

Add Log.v calls in both of these branches that logs some useful message that we're doing this.
Attachment #8730977 - Flags: review?(bugmail.mozilla) → review+
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #3)
> no expected behaviour

no *un*expected behaviour, is what I meant :)
https://hg.mozilla.org/mozilla-central/rev/c7969fc23e51
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.