Bug 32378 - CHROMIUM: Alpha blending in SVG filters is wrong
Summary: CHROMIUM: Alpha blending in SVG filters is wrong
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows 7
: P2 Normal
Assignee: Stephen White
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-10 07:54 PST by Stephen White
Modified: 2009-12-10 14:54 PST (History)
1 user (show)

See Also:


Attachments
Expected results from LayoutTests/svg/filters/feGaussianBlur.svg (40.37 KB, image/png)
2009-12-10 07:54 PST, Stephen White
no flags Details
Actual results from LayoutTests/svg/filters/feGaussianBlur.svg (24.54 KB, image/png)
2009-12-10 07:55 PST, Stephen White
no flags Details
Fix (don't apply the gamma LUT to the alpha channel). (1.51 KB, patch)
2009-12-10 08:05 PST, Stephen White
krit: review+
krit: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen White 2009-12-10 07:54:53 PST
Created attachment 44613 [details]
Expected results from LayoutTests/svg/filters/feGaussianBlur.svg

See attached.  The soft edges are more saturated than they should be.
Comment 1 Stephen White 2009-12-10 07:55:39 PST
Created attachment 44614 [details]
Actual results from LayoutTests/svg/filters/feGaussianBlur.svg
Comment 2 Stephen White 2009-12-10 08:05:35 PST
Created attachment 44615 [details]
Fix (don't apply the gamma LUT to the alpha channel).
Comment 3 WebKit Review Bot 2009-12-10 08:09:28 PST
style-queue ran check-webkit-style on attachment 44615 [details] without any errors.
Comment 4 Dirk Schulze 2009-12-10 08:51:19 PST
Comment on attachment 44615 [details]
Fix (don't apply the gamma LUT to the alpha channel).

Good catch. LGTM
Qt and Gtk need the same fix in a later patch.
You can commit the patch yourself or set the commit queue flag.
Comment 5 Stephen White 2009-12-10 14:54:46 PST
Landed as r51950.  Closing.