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

Issue 596778 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Scroll bar bounces back after scrolling page up/down for pdf file.

Reported by vku...@etouch.net, Mar 22 2016

Issue description

Chrome Version: 51.0.2687.0 (Official Build)Revision 5ee3f242cefebba7eb0f05424e7308f9158d8388-refs/heads/master@{#382459} (64-bit)

What steps will reproduce the problem?
1.Launch chrome,navigate to http://www.orimi.com/pdf-test.pdf
2.Scroll down the  page and observe.

Actual: Menu header gets transparent after scrolling down pdf page.

Expected: Menu header should not be transparent after scrolling down pdf page.


This is a regression issue broken in 'M51' and will soon update other info.
 

Comment 1 by vku...@etouch.net, Mar 22 2016

Labels: hasbisect
Owner: dtapu...@chromium.org
Status: Assigned (was: Unconfirmed)
Summary: Regression:Scroll bar bounces back after scrolling page up/down for pdf file. (was: Regression:Menu header gets transparent after scrolling down pdf page.)
Correction:
Actual: Scroll bar bounces back after scrolling page up/down for pdf file.

Expected: Scroll bar should work smoothly after scrolling page up/down for pdf file.

Manual regression range:
Good Build: 51.0.2686.0
Bad Build:  51.0.2687.0

Narrow bisect:
https://chromium.googlesource.com/chromium/src/+log/6bebadddb399ad1c1f91592efed410040617e708..5840ac85a7c912d9ddb764007b6a9ec65a26ad22?pretty=fuller&n=30

Suspecting: 382345 ?
Kindly help to re-assign, if your changes are not cause for this issue.

Note: Issue not seen on Win & Linux OS.
Actual_Scroll.mov
2.5 MB Download
Labels: ReleaseBlock-Beta
Adding release block label, please undo if not the case.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 23 2016

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

commit c1d91e8f7cd46f1cb68ec09e49d4de5c90175d6c
Author: dtapuska <dtapuska@chromium.org>
Date: Wed Mar 23 19:34:51 2016

Fix missing fields in GestureScrollBegin/Update/End event conversions.

Some of the fields were missing from the event conversion path that
occurs when a plugin is used. This caused fields to be populated
with the default value. As a result the default values enabled
overscroll scrolling when using a low precision delta
mouse wheel with an MacOSX machine.

The bug did not indicate which precision of mouse was being
used but overscrolling for PDF files with a touchpad or a magic
mouse (which have high precision deltas) still enable
elastic overscroll; but that is consistent with MacOSX.
This is a behavioural change for MacOSX but is consistent
with Safari when using a high precision mouse.

BUG= 596778 

Review URL: https://codereview.chromium.org/1829743003

Cr-Commit-Position: refs/heads/master@{#382901}

[modify] https://crrev.com/c1d91e8f7cd46f1cb68ec09e49d4de5c90175d6c/third_party/WebKit/Source/core/events/GestureEvent.cpp
[modify] https://crrev.com/c1d91e8f7cd46f1cb68ec09e49d4de5c90175d6c/third_party/WebKit/Source/core/events/GestureEvent.h
[modify] https://crrev.com/c1d91e8f7cd46f1cb68ec09e49d4de5c90175d6c/third_party/WebKit/Source/platform/PlatformGestureEvent.h
[modify] https://crrev.com/c1d91e8f7cd46f1cb68ec09e49d4de5c90175d6c/third_party/WebKit/Source/web/WebInputEventConversion.cpp
[modify] https://crrev.com/c1d91e8f7cd46f1cb68ec09e49d4de5c90175d6c/third_party/WebKit/Source/web/tests/WebInputEventConversionTest.cpp

Status: Fixed (was: Assigned)
This is fixed for non high precision mouse wheels.

If you use a touchpad it will animate overscroll much like everything else does in MacOSX.
Labels: Hotlist-Input-Dev

Sign in to add a comment