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

Issue 689472 link

Starred by 3 users
Status: WontFix
Owner:
mcnee@chromium.org
Closed: Feb 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression:Weird behavior of Scroll bar is seen after zooming out the pdf page

Reported by adha...@etouch.net, Feb 7 2017 
Chrome Version : 58.0.3004.4 (Official Build) 54ca43feba44b861ed9986bb7d3b65d7178edd41-refs/branch-heads/3004@{#5} 32/64 bit
OS : Windows (7,8,10),Linux (14.04 LTS)

What steps will reproduce the problem?
(1)Launch chrome and navigate to https://pdfobject.com/static.html
(2)Zoom out the page up to 50% via wrench menu.
(3)Observe the L.H.S of the scroll bar.(Kindly refer the video)

Actual:Weird behavior of Scroll bar is seen after zooming out the page.i.e black patch is seen

Expected:No such black patch should be seen after zooming out the page via wrench menu.

This is a Regression issue broken in M-57,will soon update other info.

Good build:57.0.2943.0
Bad build:57.0.2944.0

Note:Above issue is not seen on Mac OS.
Actual result.mp4
782 KB View Download
Expected result.mp4
676 KB View Download

Comment 1 by rbasuvula@chromium.org, Feb 7 2017

Cc: rbasuvula@chromium.org
Labels: hasbisect-per-revision ReleaseBlock-Stable
Owner: mcnee@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 57.0.2943.0 (Revision: 436483).
Bad build: 57.0.2944.0 (Revision: 436816).

You are probably looking for a change made after 436694 (known good), but no later than 436695 (first known bad).

CHANGE-LOG URL:
---------------
https://chromium.googlesource.com/chromium/src/+log/0d0973803e7802215f26802941eb4bb2f0c97037..2999413a1038cfd7a2020edf431671a17db0e8d8

From the CL above, assigning the issue to the concern owner

@mcnee:Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Review-Url: https://codereview.chromium.org/2503633002

Note : 1.Able to reproduce the issue in Ubuntu 14.04 & Win 10.0 Not in Mac 10.12.2.
2.Able to reproduce in latest Canary #58.0.3004.3
3.Adding Release Block-Stable for this issue.Please remove if not the case.

Comment 2 by gov...@chromium.org, Feb 8 2017 

A friendly reminder that M57 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!

Comment 3 by wjmaclean@chromium.org, Feb 8 2017 

govind@ - How soon is 'soon'?

mcnee@ has a fix, and it should be mergeable, but it would help to know the timelines here. The "bug" technically isn't really a regression, rather a new behaviour was introduced (and the new behaviour has a small bug). In any case, the behaviour will change.

Comment 4 by mcnee@chromium.org, Feb 8 2017

Status: Started (was: Assigned)
The behaviour was indeed changed in https://codereview.chromium.org/2503633002 where we now propagate browser zoom changes to embedded pdfs. The previous behaviour where we ignore the changes also introduced a horizontal scrollbar on zoom out since we don't change the size of the content. Now we may accidentally introduce a horizontal scrollbar due to rounding issues with the size of the embedded pdf's viewport.

Comment 5 by gov...@chromium.org, Feb 9 2017 

Re #3, if change is merged to M57 branch by next week Friday (02/17) will be great so we can pick it up for following week beta release. Please request a merge to M57 once change is landed/baked/verified in Canary. Thank you.

Comment 6 by mcnee@chromium.org, Feb 14 2017

Status: WontFix (was: Started)
After further investigation, the change to propagate zoom to embedded pdf causes this behaviour. Due to the fixed size of non-overlay scrollbars, the new content size may cause scrollbars to be introduced/removed. This is a consequence of the new behaviour and we consider this to be WAI.

When overlay scrollbars are turned on, this will no longer be an issue.
►

Sign in to add a comment