New issue
Advanced search Search tips

Issue 818873 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

scrollbar incorrectly appears next to some images at certain page zooms/system scalings

Reported by billdill...@gmail.com, Mar 5 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.186 Safari/537.36

Example URL:
https://www.reddit.com/r/pics/comments/8289m7/great_pyramid_egypt/

Steps to reproduce the problem:
there is a vertical scrollbar that appears beside some images on reddit at certain page zoom percentages. You can go through the various page zooms and see the scroll bar disappear at some percentages 

What is the expected behavior?
Probably the easiest way to reproduce this issue: set the Windows 10 system scaling to 175%. Edge and Firefox do not have this issue at any zoom level

What went wrong?
this seems similar to  issue 777095 

Does it occur on multiple sites: N/A

Is it a problem with a plugin? N/A 

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 64.0.3282.186  Channel: stable
OS Version: 10.0
Flash Version:
 
incorrectscrollbar.png
2.5 MB View Download
Components: Blink
Labels: Needs-Triage-M64

Comment 2 by tkent@chromium.org, Mar 7 2018

Components: -Blink Blink>Layout Blink>Scroll
Labels: -Pri-2 -Type-Compat hasbisect-per-revision RegressedIn-62 Target-67 Triaged-ET Target-66 M-67 FoundIn-66 FoundIn-67 Target-65 FoundIn-65 OS-Linux Pri-1 Type-Bug-Regression
Owner: ka...@opera.com
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Windows 10 and Ubuntu 14.04 using chrome reported version #64.0.3282.186 and latest canary #67.0.3364.0. Issue is not seen in OS-Mac.

Bisect Information:
=====================
Good build: 62.0.3169.0
Bad Build : 62.0.3170.0

Change Log URL: 
https://chromium.googlesource.com/chromium/src/+log/97eedd95bbffdf6b2a24494a599a5089bcad1f94..abc4634436dd4fc1b909ec04e589288464d432aa

From the above change log suspecting below change
Change-Id: I15850c55d54fbb0c885d4040be3c90ea2a51d7ca
Reviewed-on: https://chromium-review.googlesource.com/543141

karlo@ - 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.

Thanks...!!
Project Member

Comment 4 by sheriffbot@chromium.org, Mar 7 2018

Labels: Hotlist-Recharge-BouncingOwner
Owner: ----
Status: Untriaged (was: Assigned)
The assigned owner "karlo@opera.com" is not able to receive e-mails, please re-triage.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by e...@chromium.org, Mar 7 2018

Owner: mstensho@chromium.org
Status: Assigned (was: Untriaged)
More rounding fun, would you mind taking a look at this one Morten as Christian is out?
tc.html
447 bytes View Download
Further simplification. Got rid of the image and inconsistent layout. Here the scrollbar shows up consistently. Probably the same issue as the original one, but time will show.
tc-maybe.html
190 bytes View Download
similar issue here, please check it out https://bugs.chromium.org/p/chromium/issues/detail?id=807790 
tc-without-zoom.html
202 bytes View Download
I don't think bug 807790 is the same problem (but then again, I cannot reproduce that issue)
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 9 2018

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

commit 0e7d0d12eade3a80ef79e6c19a14bade77cc5b8e
Author: Morten Stenshorne <mstensho@chromium.org>
Date: Fri Mar 09 11:45:14 2018

Don't pixel-snap the top and bottom of lines.

That might cause incorrect detection of overflow, since the height of
the block isn't pixel-snapped.

Bug:  818873 
Change-Id: I864be9dfedf6f008fe19ce02088ac6488ec274d0
Reviewed-on: https://chromium-review.googlesource.com/954022
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542080}
[add] https://crrev.com/0e7d0d12eade3a80ef79e6c19a14bade77cc5b8e/third_party/WebKit/LayoutTests/external/wpt/css/CSS2/linebox/fractional-line-height.html
[modify] https://crrev.com/0e7d0d12eade3a80ef79e6c19a14bade77cc5b8e/third_party/WebKit/Source/core/layout/line/InlineBox.h
[modify] https://crrev.com/0e7d0d12eade3a80ef79e6c19a14bade77cc5b8e/third_party/WebKit/Source/core/layout/line/InlineFlowBox.cpp

Status: Fixed (was: Assigned)
this is fixed in 67?
Yeah, should be.

Sign in to add a comment