RESOLVED FIXED 7200
Gradient appears and disappears in SVG when text is off screen 
https://bugs.webkit.org/show_bug.cgi?id=7200
Summary Gradient appears and disappears in SVG when text is off screen
Alexander Kellett
Reported Saturday, February 11, 2006 11:14:44 PM UTC
scrolling off the side of http://bugzilla.opendarwin.org/attachment.cgi?id=6042 shows the issue
Attachments
Reduced testcase. (684 bytes, image/svg+xml)
2007-01-02 06:19 PST, Nikolas Zimmermann 		no flags
Details
Initial patch (83.47 KB, text/plain)
2007-09-03 14:16 PDT, Nikolas Zimmermann 		no flags
Details
Updated patch (667.28 KB, patch)
2007-09-03 15:32 PDT, Nikolas Zimmermann 		no flags
DetailsFormatted DiffDiff
Updated patch v2 (84.17 KB, patch)
2007-09-03 15:43 PDT, Nikolas Zimmermann 		oliver: review+
DetailsFormatted DiffDiff
Layout test results (574.60 KB, text/plain)
2007-09-03 15:44 PDT, Nikolas Zimmermann 		oliver: review+
Details
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Nikolas Zimmermann
Comment 1 Tuesday, January 2, 2007 2:19:56 PM UTC
Created attachment 12159 [details] Reduced testcase. The problem is actually, because of <tspan> in <text>, that breaks "gradient-on-text".
Eric Seidel (no email)
Comment 2 Wednesday, January 31, 2007 12:51:24 PM UTC
I can't tell if this is fixed. Niko?
Nikolas Zimmermann
Comment 3 Sunday, February 4, 2007 1:15:49 PM UTC
(In reply to comment #2) > I can't tell if this is fixed. Niko? > Nope, problem is still persistent. Not sure how to fix yet. Niko
Nikolas Zimmermann
Comment 4 Monday, September 3, 2007 10:16:20 PM UTC
Created attachment 16187 [details] Initial patch Not ready for review yet. Just for Oliver :-)
Nikolas Zimmermann
Comment 5 Monday, September 3, 2007 11:32:46 PM UTC
Created attachment 16188 [details] Updated patch The reduced testcase is obsolete as the problem gets highlighted by a different (already commited) batik/text testcase.
Nikolas Zimmermann
Comment 6 Monday, September 3, 2007 11:43:54 PM UTC
Created attachment 16189 [details] Updated patch v2 Some last hot fixes - demanded by Oliver. Seperated LayoutTests/ and patch upon request.
Nikolas Zimmermann
Comment 7 Monday, September 3, 2007 11:44:59 PM UTC
Created attachment 16190 [details] Layout test results
Oliver Hunt
Comment 8 Tuesday, September 4, 2007 9:30:16 AM UTC
Comment on attachment 16189 [details] Updated patch v2 Basically looks good, but i find myself wondering about SVGTextChunkWalkerBase and SVGTextChunkWalker, as it seems unnecessary to use both virtual methods *and* function pointers.
Oliver Hunt
Comment 9 Tuesday, September 4, 2007 9:36:23 AM UTC
i'll r+ this if you can justify the exciting combo of function pointer and virtual methods in SVGTextChunkWalkerBase
Nikolas Zimmermann
Comment 10 Tuesday, September 4, 2007 12:43:32 PM UTC
(In reply to comment #9) > i'll r+ this if you can justify the exciting combo of function pointer and > virtual methods in SVGTextChunkWalkerBase Hey Oliver, yeah nice combination :-) In fact I added SVGTextChunkWalkerBase later on, as it made it possible to remove the template based implementation of "walkTextChunks" in SVGRootInlineBox.h - I didn't like the fact to have crucial methods sitting in header files. Greetings, Niko
Oliver Hunt
Comment 11 Wednesday, September 5, 2007 7:20:25 PM UTC
Comment on attachment 16190 [details] Layout test results changes look sane, rubber stamping
Nikolas Zimmermann
Comment 12 Wednesday, September 5, 2007 7:29:21 PM UTC
Landed in r25370.
Note You need to log in before you can comment on or make changes to this bug.