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

Issue 673597 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Document element cannot always omit isolation

Project Member Reported by trchen@chromium.org, Dec 13 2016

Issue description

The previous assumption that nothing can be the backdrop of the document element's stacking context was incorrect, because document background doesn't paint as a part of the stacking context but as canvas backdrop instead.

Note that only non-composited blending elements are affected by this bug. We should match composited code path, which also matches Firefox behavior.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 13 2016

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

commit cf97fd35cae7ca6fae58cff6eea2f6cc59716171
Author: trchen <trchen@chromium.org>
Date: Tue Dec 13 21:53:37 2016

[Blink>Paint] Document element cannot always omit isolation

The previous assumption that nothing can be the backdrop of the document
element's stacking context was incorrect, because document background doesn't
paint as a part of the stacking context but as canvas backdrop instead.

Note that only non-composited blending elements are affected by this bug.
This CL changes behavior to match composited code path, which also matches
Firefox behavior.

BUG= 673597 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2564423003
Cr-Commit-Position: refs/heads/master@{#438298}

[modify] https://crrev.com/cf97fd35cae7ca6fae58cff6eea2f6cc59716171/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/cf97fd35cae7ca6fae58cff6eea2f6cc59716171/third_party/WebKit/LayoutTests/css3/blending/mix-blend-mode-background-size.html
[modify] https://crrev.com/cf97fd35cae7ca6fae58cff6eea2f6cc59716171/third_party/WebKit/LayoutTests/platform/linux/css3/blending/mix-blend-mode-simple-text-expected.png
[modify] https://crrev.com/cf97fd35cae7ca6fae58cff6eea2f6cc59716171/third_party/WebKit/LayoutTests/platform/win/css3/blending/mix-blend-mode-simple-expected.png
[modify] https://crrev.com/cf97fd35cae7ca6fae58cff6eea2f6cc59716171/third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp

Status: Fixed (was: Started)

Sign in to add a comment