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

Issue 802915 link

Starred by 1 user
Status: WontFix
Owner: ----
Closed: Jan 18
Cc:
█ trchen@chromium.org
wangxianzhu@chromium.org
schenney@chromium.org
chrishtr@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[SPv1] Scrolling contents layer should not isolate children as a side effect

Project Member Reported by trchen@chromium.org, Jan 17 2018 
See the attached layout test.

This bug is discovered while investigating  issue 798148 , https://bugs.chromium.org/p/chromium/issues/detail?id=798148#c6.

The creation of scrolling contents layer has a side effect of isolating its descendant layers. As a result, if there are any non-local background that are promoted into a background layer, the background will be outside of that isolated group, thus blending descendants won't blend with the background properly.

It would be pretty difficult to fix in SPv1/SPv175, while SPv2 is immune to it.
test20.html
276 bytes View Download
test20-expected.html
228 bytes View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 17 2018 

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

commit f758f57f0648b5aa316e7cdd23f95916e1c55995
Author: Tien-Ren Chen <trchen@chromium.org>
Date: Wed Jan 17 18:13:31 2018

[Blink] Add layout test for  issue 802915 

In SPv1 we don't handle isolation correctly for elements with non-local
background and impl-side scrolling at the same time. It is hard to fix
in our current architecture thus only add a test to keep track of it.

BUG= 802915 

Change-Id: Ib152fe3961e490ecb800ca912f56c8bcfa02021f
Reviewed-on: https://chromium-review.googlesource.com/868399
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529806}
[modify] https://crrev.com/f758f57f0648b5aa316e7cdd23f95916e1c55995/third_party/WebKit/LayoutTests/TestExpectations
[add] https://crrev.com/f758f57f0648b5aa316e7cdd23f95916e1c55995/third_party/WebKit/LayoutTests/css3/blending/isolation-should-include-non-local-background-expected.html
[add] https://crrev.com/f758f57f0648b5aa316e7cdd23f95916e1c55995/third_party/WebKit/LayoutTests/css3/blending/isolation-should-include-non-local-background.html
Project Member

Comment 2 by sheriffbot@chromium.org, Jan 18 (4 days ago)

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

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

Comment 3 by wangxianzhu@chromium.org, Jan 18 (4 days ago)

Status: WontFix (was: Untriaged)
The test is passing on CompositeAfterPaint. We won't fix this issue for pre-CompositeAfterPaint.
►

Sign in to add a comment