New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 783798 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 785963
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression

Blocking:
issue 672921



Sign in to add a comment

overscroll-behavior bounces background

Project Member Reported by brentons@google.com, Nov 10 2017

Issue description

Chrome Version: 64.0.3263.0 (Official Build) canary (64-bit)
OS: macOS 10.12.6

What steps will reproduce the problem?
(1) Navigate to https://ebidel.github.io/demos/chatbox.html
(2) Scroll past the top of the chat box.

What is the expected result?
Nothing - the scroll should only affect the chat box.

What happens instead?
If the background is at scrollY=0, it will overscroll the background (causing the viewport to bounce).
 
overscroll-behavior-failure.mov
4.6 MB Download
Components: -Blink>CSS
Labels: Needs-Bisect
Tested on Windows 10:

Chrome Stable 62.0.3202.89 (Official Build) (64-bit) (cohort: 62_89_win): BAD
Chrome Canary 64.0.3266.0 (Official Build) canary (32-bit) (cohort: Clang-32): GOOD

Requesting reverse bisect.
Cc: sunyunjia@chromium.org susanjuniab@chromium.org
Labels: -Type-Bug -Pri-3 -Needs-Bisect ReleaseBlock-Stable Needs-Triage-M64 M-62 OS-Linux OS-Windows Pri-1 Type-Bug-Regression
Owner: majidvp@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce this issue on Mac OS 10.12.6, Windows 7 and Ubuntu 14.04 on latest stable 62.0.3202.89 with the steps mentioned in the original comment.
Issue is not observed on latest Canary 64.0.3266.0, Dev 64.0.3260.2 and Beta 63.0.3239.40.

Performed Reverse Bisect and below is the bisect information.

Reverse Bisect Information:
=====================
Good build: 63.0.3239.27 
Bad Build : 63.0.3239.26 

As both good and bad are branch builds, unable to run the per-revision bisect script. Hence providing the Change log URL through Manual Bisect from Omahaproxy .

Change Log URL: 
===============
https://chromium.googlesource.com/chromium/src/+log/63.0.3239.26..63.0.3239.27?pretty=fuller&n=10000

From the above change log the possible CL suspect is:
Reviewed-on: https://chromium-review.googlesource.com/740061

majidvp@ Could you please check whether this issue is caused with respect to your change, if not please help us in assigning it to the right owner.

Adding ReleaseBlock-Stable label as this issue is seen on Stable build. Please feel free to remove the same if it is not applicable.


Thanks...

Comment 3 by brentons@google.com, Nov 13 2017

Summary: overscroll-behavior bounces background (was: overscroll-behavior bounds background)

Comment 4 by brentons@google.com, Nov 13 2017

I don't believe this feature was introduced until 63:

https://www.chromestatus.com/features/5734614437986304
Cc: -sunyunjia@chromium.org majidvp@chromium.org
Labels: -Pri-1 -ReleaseBlock-Stable -M-62 M-63 Pri-2
Owner: sunyunjia@chromium.org
Overscroll-behavior does not exist in M62. So this only affects M63.
Also I don't think this is a major issue to be a release blocker. So removing
that label and making this s P2.


Assigning to sunyunjia@ to investigate.


Can this be related to how we 
Labels: allpublic
BTW, I removed the google restricted view since it is not really necessary. I think it was added since the bug was filed with @google.com email address.
Labels: -allpublic
Labels: allpublic
Status: WontFix (was: Assigned)
Looking at this change log: https://chromium.googlesource.com/chromium/src/+log/63.0.3239.26..63.0.3239.27?pretty=fuller&n=10000

A renaming from scroll-boundary-behavior to overscroll-behavior happens during this period. So this should just a issue due to renaming. The renaming has been merged to M63, so the problem won't really affect its being shipped.
Status: Assigned (was: WontFix)
That video clip shows the demo page from ebidelman@. I am pretty sure that uses
the correct property name. Also it shows the property working in some situation
but not the other. If this was a naming issue then it wouldn't work at all.

I think the bisect in #2 is probably not accurate because you can recreate the
problem even when the feature is not available (pre M62 or early M63 before rename). 
So it can lead to incorrect bad revision.

sunyunjia@: have you tried recreating the issue yourself on Canary?

 
Status: Started (was: Assigned)
Sorry my bad. The background won't bounce if it is a touch-move scroll, but will bounce if it is a fling scroll. Let me take a look at it.
Mergedinto: 785973
Status: Duplicate (was: Started)
Mergedinto: -785973 785963
Blocking: 672921

Sign in to add a comment