Issue metadata
Sign in to add a comment
|
Nested contain: paint breaks scrolling invalidation on high-DPI displays
Reported by
a...@scirra.com,
Sep 18 2016
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2864.0 Safari/537.36 Example URL: https://dl.dropboxusercontent.com/u/15217362/bugs/contain-paint-bug-2.html Steps to reproduce the problem: 1. Visit the given URL on a system with high-DPI display (devicePixelRatio > 1) 2. Scroll down in the scrolling div What is the expected behavior? Content to display normally What went wrong? As you scroll down, content outside the initial viewport is not painted. It reappears if you select text. Both the scrolling div and the content that disappears uses contain: paint. Without this it displays correctly. On lo-DPI displays (DPR = 1) it also displays correctly. Does it occur on multiple sites: N/A Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? N/A Chrome version: 55.0.2864.0 Channel: canary OS Version: 10.0 Flash Version: Shockwave Flash 22.0 r0 The issue and repro is similar to bug 606342 .
,
Sep 20 2016
Reproduces on Android and Mac and reproduces with --enable-prefer-compositing-to-lcd-text. Indeed seems essentially the same issue as bug 606342 . Thanks for telling us about that.
,
Sep 20 2016
,
Oct 5 2016
Now also reproduces on Canary with low-DPI display.
,
Oct 7 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/56d8f0dc244d264c3a99d13b6b06f143a81c06d2 commit 56d8f0dc244d264c3a99d13b6b06f143a81c06d2 Author: wangxianzhu <wangxianzhu@chromium.org> Date: Fri Oct 07 02:45:09 2016 Don't clip contain:paint when requested to ignore clip This fixes another place that previously forced clip for contain:paint, causing corrupted painting in nested contain:paint containers. Other places were fixed in https://codereview.chromium.org/1985933002. BUG= 648091 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://codereview.chromium.org/2400063002 Cr-Commit-Position: refs/heads/master@{#423788} [modify] https://crrev.com/56d8f0dc244d264c3a99d13b6b06f143a81c06d2/third_party/WebKit/Source/core/paint/PaintLayerClipper.cpp [modify] https://crrev.com/56d8f0dc244d264c3a99d13b6b06f143a81c06d2/third_party/WebKit/Source/core/paint/PaintLayerClipper.h [modify] https://crrev.com/56d8f0dc244d264c3a99d13b6b06f143a81c06d2/third_party/WebKit/Source/core/paint/PaintLayerClipperTest.cpp
,
Oct 7 2016
,
Jan 17 2017
The fix to this bug missed the M55 (@ chromium rev 423768) by 20 revisions. Marking as an M55 regression. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dtapu...@chromium.org
, Sep 20 2016