New issue
Advanced search Search tips

Issue 832547 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression

Blocked on:
issue 659750



Sign in to add a comment

Regression:Background page gets scroll up after auto-scrolling inside the pdf file.

Reported by vku...@etouch.net, Apr 13 2018

Issue description

Chrome Version:67.0.3396.0 (Official Build)Revision 9ef2aa869bc7bc0c089e255d698cca6e47d6b038-refs/heads/master@{#550428} (32/64-bit)
OS:Windows(7,8,8.1,10)

What steps will reproduce the problem?
(1)Launch chrome and navigate to https://msu.edu/~urban/sme865/resources/embedded_pdf.html
(2)Scroll the pdf file using middle mouse wheel down then upward and observe 

Actual: Background page gets scroll up after auto-scrolling inside the pdf file.

Expected: Background page should not scroll up after auto-scrolling inside the pdf file.

This is a regression issue broken in 'M67' and below is the manual bisect info
Good Build:  67.0.3382.0(Revision:546347)
Bad Build:   67.0.3383.0(Revision:546671)

Note: Issue is not reproducible on Mac(10.12.6, 10.13.1, 10.13.5) and Linux(14.04 LTS) OS
 
Actual_pdf.mp4
1.7 MB View Download
Expected_pdf.mp4
1.0 MB View Download

Comment 1 by vku...@etouch.net, Apr 13 2018

Labels: hasbisect-per-revision RegressedIn-67 Target-67 FoundIn-67
Owner: sahel@chromium.org
Status: Assigned (was: Unconfirmed)
You are probably looking for a change made after 546635 (known good), but no later than 546636 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.
https://chromium.googlesource.com/chromium/src/+log/ed8f674f763a5eaba8c31d417cc13d7190127186..46259cc58e11f12ca3308b694c59a14b195fc703

Suspect: https://chromium.googlesource.com/chromium/src/+/46259cc58e11f12ca3308b694c59a14b195fc703

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

Comment 2 by sahel@chromium.org, May 4 2018

Blockedon: 659750
Labels: -Pri-1 Pri-2
With Autoscroll browser side fling, the renderer handles GSUs from autoscroll similar to GSUs from touchscreen and wheel scrollling.

If you try wheel/touchscreen scroll you will notice that the scroll bubbles to the parent once the pdf scroller hits its extent.

The root cause of the issue is that pdf is currently using BrowserPlugin instead of oopif, and BrowserPlugin doesn't support scroll latching.

I'll reduce the priority of this bug, since even though it is a recent regression in autscroll behavior, wheel and touchscreen scrolling has been always like this and once pdf switches to using oopif the scroll latching will work for all types of scrolling.
MimeHandlerViewGuest embeds another plugin inside (PDF PPAPI). I wonder if same problem exists there as well. If so, even with MimeHandlerViewGuest transitioning to using OOPIF this bug won't be resolved. But perhaps we could look into that "after" MimeHandlerViewGuest moved to using OOPIFs but that will won't be any time soon.

Comment 4 by sahel@chromium.org, May 4 2018

Had a chat with ekaramad@, we are hoping that crbug.com/659750 will be fixed by the end of Q3.
Since other kinds of scrolling has had this behavior for a while (touchscreen scrolling since M47 and wheel scrolling since M65), I think waiting till Q3 is fine.

Comment 5 by sahel@chromium.org, May 4 2018

Labels: -M-67 -Target-67 Target-70

Sign in to add a comment