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

Issue 707455 link

Starred by 2 users
Status: Fixed
Owner:
█ trchen@chromium.org
Last visit > 30 days ago
Closed: Apr 2017
Cc:
pdr@chromium.org
chrishtr@chromium.org
vmp...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[SPv2] Application order between CSS clip and stacking context effects is wrong

Project Member Reported by trchen@chromium.org, Apr 1 2017 
SPv2 applies CSS clip before stacking context effects.

SPv1 & IE & Edge applies CSS clip after.
Firefox applies CSS clip both before and after.
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 3 2017 

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

commit 6b2de0f362490dc4823e64976d9682c37a62c64d
Author: trchen <trchen@chromium.org>
Date: Mon Apr 03 21:14:33 2017

[SPv2] Switch the application order between effects and CSS clip

According to SVG spec, clipping, masking, and opacity applies after
filter. The CSS version follows the same order. This is also what SPv1
and other vendors implemented.

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

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

[add] https://crrev.com/6b2de0f362490dc4823e64976d9682c37a62c64d/third_party/WebKit/LayoutTests/fast/clip/css-clip-filter-order-expected.html
[add] https://crrev.com/6b2de0f362490dc4823e64976d9682c37a62c64d/third_party/WebKit/LayoutTests/fast/clip/css-clip-filter-order.html
[modify] https://crrev.com/6b2de0f362490dc4823e64976d9682c37a62c64d/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp

Comment 2 by trchen@chromium.org, Apr 3 2017

Status: Fixed (was: Assigned)

Comment 3 by schenney@chromium.org, Apr 4 2017

Labels: BugSource-Team PaintTeamTriaged-20170401
►

Sign in to add a comment