Skip to content

Commit

Permalink
PixelMovingFilterClipExpander in blink paint property tree
Browse files Browse the repository at this point in the history
It translates to a cc clip node with pixel_moving_filter_id to
expand the visual rect under a pixel-moving filter, to ensure all
pixels contributing to a pixel-moving filter are included before
applying the parent clip on the result.

For now this node doesn't contribute to geometry mapping in blink,
though it affects layerization results if the clip state of a
PendingLayer is changed by the existence of the node.

In the future we can use this node to simplify geometry mapping
with pixel-moving filters, or let PropertyTreeManager automatically
generate cc clip nodes for pixel-moving filters without this blink
paint property node.

Bug: 1228817
Change-Id: I38dfcbc4e7a3894b0f8ea66775e0ad5178b0099c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3742098
Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1024957}
  • Loading branch information
wangxianzhu authored and chromium-wpt-export-bot committed Jul 15, 2022
1 parent 77d05b2 commit bc4ffb9
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions css/filter-effects/will-change-blur-filter-under-clip.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
<!DOCTYPE html>
<link rel="help" href="https://drafts.fxtf.org/filter-effects-1/#FilterProperty">
<link rel="help" href="https://crbug.com/1228817">
<link rel="match" href="reference/green-100x100.html">
<div style="width: 100px; height: 100px; overflow: hidden">
<div style="width: 200px; height: 200px; margin-top: -50px; margin-left: -50px;
background: green; filter: blur(10px); will-change: filter"></div>
</div>

0 comments on commit bc4ffb9

Please sign in to comment.