New issue
Advanced search Search tips

Issue 758920 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Unwanted scrollbar on chrome://settings/URLs when page zoomed to 125%

Project Member Reported by ajha@chromium.org, Aug 25 2017

Issue description

Chrome Version: 62.0.3196.0
OS: Windows-10,Linux Ubuntu 14.04

What steps will reproduce the problem?
(1) Launch chrome and go to chrome://settings. Zoom the page to 125% 
(2) Now open Manage search engine or chrome://settings/startupPages and observe.


What is the expected result?
>No vertical scroll bar should be seen. 

What happens instead?
>Unwanted vertical scroll bar seen on the page. 

Please use labels and text to provide additional information.
This is regression issue broken in M-62 and Issue is working fine on Mac OS 10.12.6 laptop.

 
VerticalScrollBar_Actual.png
91.9 KB View Download
VerticalScrollBar_Expected.png
89.4 KB View Download

Comment 1 by ajha@chromium.org, Aug 25 2017

Cc: skobes@chromium.org
Labels: hasbisect-per-revision
Owner: szager@chromium.org
Status: Assigned (was: Untriaged)
Last good build: 62.0.3178.0
First bad build: 62.0.3179.0

Changelog:
=========
https://chromium.googlesource.com/chromium/src/+log/8ae3ea1c2efcf9d2786cb1af66315b08f0a95051..01cf6fd553bcc6d5a9ca6763eed42fbca5ecc494

szager@: Could you please take a look at this.

Thank you!
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 31 2017

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

commit 645fad1cc5e19d9d002bf128124e61453c4411c0
Author: Stefan Zager <szager@chromium.org>
Date: Thu Aug 31 23:39:08 2017

Use LayoutUnits when computing scrollbar existence.

Before this patch, a non-pixel-aligned box with overflow:auto could get spurious
scrollbars.

BUG= 758920 

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ifeba759baa2a2503b12673e53aba0dfe55cb5510
Reviewed-on: https://chromium-review.googlesource.com/639454
Commit-Queue: Stefan Zager <szager@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499071}
[add] https://crrev.com/645fad1cc5e19d9d002bf128124e61453c4411c0/third_party/WebKit/LayoutTests/fast/sub-pixel/block-with-border-overflow-expected.html
[add] https://crrev.com/645fad1cc5e19d9d002bf128124e61453c4411c0/third_party/WebKit/LayoutTests/fast/sub-pixel/block-with-border-overflow.html
[modify] https://crrev.com/645fad1cc5e19d9d002bf128124e61453c4411c0/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
[modify] https://crrev.com/645fad1cc5e19d9d002bf128124e61453c4411c0/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h

Status: Fixed (was: Assigned)
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-62; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-62 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
This seems to be in M62:
https://chromium.googlesource.com/chromium/src.git/+log/refs/branch-heads/3202?n=1000

Removing Merge-TBD. 

Sign in to add a comment