Bug 14240

Summary:

Opacity affects clipping of circles

Product:

WebKit

Reporter:

Jean-Pierre Fiset <bugzilla>

Component:

SVG

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WORKSFORME

Severity:

Normal

CC:

ahayes, bugzilla, jeffschiller, rubys, rwlbuis, simon.fraser

Priority:

Version:

523.x (Safari 3)

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
Test Case	none
First attempt	mjs: review-

Jean-Pierre Fiset

Reported 2007-06-20 11:59:41 PDT

I have come across a peculiar problem where a circle is not rendered correctly due to its opacity. If the opacity of the circle is 1.0, then it is fully painted. If the circle is given fractional opacity, then part of the circle is clipped. Other attributes that appear to trigger this unexpected behaviour is the radius of the circle, a fractional scaling factor and the number of circles drawn. I have come across this situation inadvertently and can not describe the situation better. I am submitting a test case along.

Attachments
Test Case (1.35 KB, image/svg+xml) 2007-06-20 12:00 PDT, Jean-Pierre Fiset	no flags	Details
First attempt (259.88 KB, patch) 2007-07-27 01:05 PDT, Rob Buis	mjs: review-	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Jean-Pierre Fiset

Comment 1 2007-06-20 12:00:42 PDT

Created attachment 15141 [details] Test Case

Rob Buis

Comment 2 2007-07-27 01:05:43 PDT

Created attachment 15698 [details] First attempt This should fix it, though it should be a temporary fix (until enclosingIntRect is fixed). Cheers, Rob.

mitz

Comment 3 2007-08-18 00:57:29 PDT

Comment on attachment 15698 [details] First attempt There are multiple uses of enclosingIntRect(const FloatRect&) in SVG code, so I am suspicious of a patch that targets just one of them. I think fixing enclosingIntRect would be much better.

Maciej Stachowiak

Comment 4 2007-08-19 12:50:24 PDT

I agree, enclosingIntRect should be fixed instead. The comment there claims there is no ceilf on win32 but that seems to be wrong as far as I can tell: <http://msdn2.microsoft.com/en-us/library/atdhw2dx(VS.80).aspx>. Even on platforms that legitimately lack ceilf it would be better to use ceil than to round. r- to consider fix to enclosingIntRect instead.

Eric Seidel (no email)

Comment 5 2007-12-27 01:47:04 PST

Sad to see this patch rotting in bugzilla. If all that's needed is a ceilf in enclosingIntRect, I would expect that would be easy to fix (and could be related to bug 16015 as well?)

Amos Hayes

Comment 6 2008-05-28 13:19:12 PDT

This is still a problem in Safari Version 3.1.1 (5525.18) Any updates?

Sam Ruby

Comment 7 2009-03-23 23:04:55 PDT

Suspiciously similar issue: http://intertwingly.net/blog/2009/03/23/SVG-Interop Related?

Simon Fraser (smfr)

Comment 8 2009-07-07 23:06:07 PDT

Seems to work fine in recent builds.

Note You need to log in before you can comment on or make changes to this bug.