New issue
Advanced search Search tips

Issue 835289 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Navigating back in history doesnt stop fling

Project Member Reported by bokan@chromium.org, Apr 20 2018

Issue description

Chrome Version       : 68.0.3400.0
OS Version: Android
URLs (if applicable) : any

What steps will reproduce the problem?
1. Find a long page 
2. Navigate to another long page 
3. Fling down. While the page is scrolling, hit back button

What is the expected result?
Previous page loads and remains stationary 


What happens instead of that?
Previous page loads but continues to scroll as the fling is transferred to the new page

Please provide any additional information below. Attach a screenshot if
possible.

UserAgentString: Mozilla/5.0 (Linux; Android 8.1.99; Build/PPP2.180412.003) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3400.0 Mobile Safari/537.36



 

Comment 1 by sahel@chromium.org, Jun 14 2018

Cc: bokan@chromium.org sahel@chromium.org
 Issue 852793  has been merged into this issue.

Comment 2 by mcnee@chromium.org, Jun 14 2018

Labels: -Type-Bug RegressedIn-67 Type-Bug-Regression
Adding info from duplicate bug.

Regression range:
Good 66.0.3359.158
Bad 67.0.3396.87
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/81c88f718627b281027e55c48504e1b8f84b7d47

commit 81c88f718627b281027e55c48504e1b8f84b7d47
Author: Sahel Sharify <sahel@chromium.org>
Date: Mon Oct 01 22:07:02 2018

Flinging stops after navigation.

This cl calls stopFling after navigation to make sure that flinging does
not continue after navigation when the new page is using the same renderer.

Bug:  835289 
Test: BrowserSideFlingBrowserTest.FlingingStopsAfterNavigation
Change-Id: I9f43689ec375605ab1c3ba38adf1df9d46b3efa5
Reviewed-on: https://chromium-review.googlesource.com/1244374
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: Sahel Sharify <sahel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595560}
[modify] https://crrev.com/81c88f718627b281027e55c48504e1b8f84b7d47/content/browser/renderer_host/input/fling_browsertest.cc
[modify] https://crrev.com/81c88f718627b281027e55c48504e1b8f84b7d47/content/browser/renderer_host/render_widget_host_impl.cc

Status: Fixed (was: Assigned)

Sign in to add a comment