New issue
Advanced search Search tips

Issue 736173 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Gmail scrolling not clipped

Project Member Reported by chrishtr@chromium.org, Jun 23 2017

Issue description

Chrome 61 bug.

1. Load mail.google.com
2. Open certain scrolling email threads
3. Scroll

Expected: scroll is clipped
Actual: scroll is not clipped and scrolls up off the page

I bisected this to this range:

https://chromium.googlesource.com/chromium/src/+log/236ebc6900f1acf1d197537bf222c72992b97f1e..6f8ac625db8fc02373327a9957e6633b62717d0a

I think it was broken by this commit:

https://chromium.googlesource.com/chromium/src/+/351b6fecd66235ac6e00e873266da5be47bf370e
 
Labels: PaintTeamTriaged-20170623 BugSource-Team
I can repro on linux with --enable-prefer-compositing-to-lcd-text
and --force-device-scale-factor=2

Fix in review.
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 26 2017

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

commit a348403ddc19ee7bb93c38e6dda500c54d12e9d6
Author: Chris Harrelson <chrishtr@chromium.org>
Date: Mon Jun 26 22:39:53 2017

Notify PaintLayer if ScrollsOverflow changed.

Bug:  736173 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ibafd4af308bb6eca6496d3c95afd8697b767185e
Reviewed-on: https://chromium-review.googlesource.com/547366
Commit-Queue: Chris harrelson <chrishtr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#482441}
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/FlagExpectations/enable-slimming-paint-v2
[add] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/flag-specific/enable-slimming-paint-v2/platform/linux/fast/overflow/overflow-with-local-background-attachment-expected.png
[add] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/flag-specific/enable-slimming-paint-v2/platform/linux/paint/invalidation/selection/selection-in-composited-scrolling-container-expected.txt
[add] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/flag-specific/enable-slimming-paint-v2/platform/linux/paint/invalidation/selection/selection-in-non-composited-scrolling-container-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/hittesting/inner-border-radius-hittest-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/linux/fast/overflow/overflow-float-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/linux/fast/overflow/overflow-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/linux/fast/overflow/overflow-with-local-background-attachment-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/linux/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/linux/virtual/disable-spinvalidation/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/css3/flexbox/flexbox-baseline-expected.png
[add] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/fast/overflow/overflow-float-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/fast/overflow/overflow-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/fast/overflow/overflow-with-local-background-attachment-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/paint/invalidation/selection/selection-in-non-composited-scrolling-container-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/virtual/disable-spinvalidation/paint/invalidation/selection/selection-in-non-composited-scrolling-container-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac-mac10.9/virtual/disable-spinvalidation/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/css3/flexbox/flexbox-baseline-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/fast/overflow/overflow-float-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/fast/overflow/overflow-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/paint/invalidation/selection/selection-in-non-composited-scrolling-container-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/virtual/disable-spinvalidation/paint/invalidation/selection/selection-in-non-composited-scrolling-container-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/mac/virtual/disable-spinvalidation/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/win/fast/overflow/overflow-float-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/win/fast/overflow/overflow-stacking-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/win/fast/overflow/overflow-with-local-background-attachment-expected.png
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/win/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/LayoutTests/platform/win/virtual/disable-spinvalidation/paint/invalidation/textarea-caret-expected.txt
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/Source/core/paint/PaintLayer.cpp
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/Source/core/paint/PaintLayer.h
[modify] https://crrev.com/a348403ddc19ee7bb93c38e6dda500c54d12e9d6/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp

Status: Fixed (was: Assigned)
Is this issue refers to the behavior similar to shown one in the attached snapshot? 


chrome_bug.png
55.6 KB View Download
Re comment 5: perhaps. Please file a new issue with exact steps to reproduce your
issue, and we will investigate.

Sign in to add a comment