Bug 45042
| Summary: | [cairo] Include support for rotations to the GraphicsContext::drawTiledShadow | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Alejandro G. Castro <alex> |
| Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | bruno.abinader, d-r, igor.oliveira, laszlo.gombos, mrobinson, ostap73, simon.fraser |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | PC | ||
| OS: | Linux | ||
| Bug Depends on: | |||
| Bug Blocks: | 90082 | ||
Alejandro G. Castro
Currently drawTiledShadow can just be used if the GraphicContext is not rotated, there are subpixel problems in the tiled regions.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Dominik Röttsches (drott)
Alejandro, do you have any example or screenshot of what's going wrong? Or any test case names that are affected by this?
Alejandro G. Castro
(In reply to comment #1)
> Alejandro, do you have any example or screenshot of what's going wrong? Or any test case names that are affected by this?
Sorry for the late answer, I'm not sure about the current situation of this issue. This bug comes from when we initially implemented tiling in ContextShadow. I thought Simon added a fix to this problem in ShadowBlur, that replaced ContextShadow but I've just checked the condition is still in the code. Basically the tiling when rotated showed some visual artifacts, you can test it forcing tiling (big shadowed rect), adding some rotation and commenting the lines that avoids the tiling in these cases in ShadowBlur.cpp.
Adding Sam so he can add information about current situation.
I hope this helps.
Alejandro G. Castro
Currently this does not make sense anymore.