Bug 127350 – [CSS Background Blending] -webkit-background-blend-mode fails for certain SVG files

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Mihai Tica]

-webkit-background-blend-mode fails for certain SVG background layers
Related sample can be found at:
http://codepen.io/mihait/pen/pFCtH

What happens is that if the background layer has a blend mode attached, it seems that the blend mode is inherited by the SVG graphicsContext when performing its painting.

[Mihai Tica]

Created attachment 221853 [details]
Patch V1

[Dirk Schulze]

Comment on attachment 221853 [details]
Patch V1

View in context: https://bugs.webkit.org/attachment.cgi?id=221853&action=review

snippets. Otherwise great.

> Source/WebCore/ChangeLog:10
> +        [CSS Background Blending] Background blending works
> +        incorrectly for certain SVG background layers: the
> +        graphics context of the SVG inherits the blend mode set
> +        on the background layer. Fix consists in drawing the SVG
> +        in a transparency layer.
> +        https://bugs.webkit.org/show_bug.cgi?id=127350
> +
> +        Reviewed by NOBODY (OOPS!).

The style for changelegs are:

date  name  email

Bug title
bug reference

reviewed by

fix description.

tests

* changed files

> Source/WebCore/ChangeLog:15
> +        (WebCore::SVGImage::draw): begin a transparency layer if a blend mode is set.

s/begin/Begin/

> LayoutTests/css3/compositing/background-blend-mode-svg.html:12
> +        -webkit-background-blend-mode: multiply, normal;

One value should be enough here, right?

[Mihai Tica]

> > LayoutTests/css3/compositing/background-blend-mode-svg.html:12
> > +        -webkit-background-blend-mode: multiply, normal;
> 
> One value should be enough here, right?

It is, however, according to the spec, there should be a blend mode specified per background layer.

[Mihai Tica]

Created attachment 221858 [details]
Patch V2

[WebKit Commit Bot]

Comment on attachment 221858 [details]
Patch V2

Clearing flags on attachment: 221858

Committed r162517: <http://trac.webkit.org/changeset/162517>