| Summary: | CSS style 'filter: drop-shadow...' does not render correctly without repaint. | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Drew McMurry <andrewmcmurry> | ||||||||||
| Component: | CSS | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||||||
| Status: | ASSIGNED --- | ||||||||||||
| Severity: | Normal | CC: | changseok, dino, esprehn+autocc, ews-watchlist, filipe.norte, glenn, Hironori.Fujii, karlcow, kondapallykalyan, magomez, pdr, pedropaulosuzuki1, remi.kalbe, simon.fraser, sisk, webkit-bug-importer, zalan | ||||||||||
| Priority: | P2 | Keywords: | BrowserCompat, InRadar | ||||||||||
| Version: | Safari 13 | ||||||||||||
| Hardware: | Mac | ||||||||||||
| OS: | macOS 10.15 | ||||||||||||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=207421 https://bugs.webkit.org/show_bug.cgi?id=236974 https://bugs.webkit.org/show_bug.cgi?id=236800 https://bugs.webkit.org/show_bug.cgi?id=236758 https://bugs.webkit.org/show_bug.cgi?id=118411 https://bugs.webkit.org/show_bug.cgi?id=245773 https://bugs.webkit.org/show_bug.cgi?id=255201 https://bugs.webkit.org/show_bug.cgi?id=258160 https://bugs.webkit.org/show_bug.cgi?id=261442 |
||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Drew McMurry
2020-02-11 14:38:22 PST
*** Bug 234639 has been marked as a duplicate of this bug. *** *** This bug has been marked as a duplicate of bug 202472 *** Reopening to track the non-transition repaint issues with drop-shadow. Created attachment 453998 [details]
Patch
Created attachment 454007 [details]
Patch
Comment on attachment 454007 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=454007&action=review > Source/WebCore/rendering/RenderBox.cpp:4998 > + overflowMinX = std::min(overflowMinX, borderBox.x() - ((!isFlipped || isHorizontal) ? filterOutsets.left() : filterOutsets.right())); "!isFlipped || isHorizontal" is a bit odd as horizontal could be flipped too (horizontal-bt, though I don't think any language uses it)so this essentially only flips for vertical writing mode. Created attachment 454625 [details]
Patch
Affects these tests: css3/filters/effect-combined-hw.html css3/filters/effect-drop-shadow-hw.html css3/filters/effect-drop-shadow.html *** Bug 245773 has been marked as a duplicate of this bug. *** Folks, what's the plan for this bug? I've tested the patch and it seems to work properly. The new tests fail on the bots cause they are missing expectations for those platforms (as you already know). Is this going to be committed? or is it breaking something that's not mentioned here? (In reply to Miguel Gomez from comment #11) > Folks, what's the plan for this bug? I've tested the patch and it seems to > work properly. The new tests fail on the bots cause they are missing > expectations for those platforms (as you already know). Is this going to be > committed? or is it breaking something that's not mentioned here? Also, anything I can do to help? Folks, I was playing with the patch provided here, and I found a case where this doesn't seem to fix the wrong rendering. I've tested that the rendering is not correct even on macOS Safari 15.4. The test is this: https://certification.bbctvapps.co.uk/act/test/ui/187/css-supports-shadows/ The image should have a drop-shadow filter that's not properly rendered. On GTK and WPE it's not rendered at all, with or without the patch here. On Safari some shadow is rendered, but it's not correct. Right, the patch is not complete (it doesn't handle invalidation on child elements correctly). It looks like bug#118411. Pull request: https://github.com/WebKit/WebKit/pull/12797 Hello This issue is affecting us. Was wondering if anyone is looking further into this? Thanks |