RESOLVED FIXED 4155
Temporary WebCore SPI necessary for SVG testing tools
https://bugs.webkit.org/show_bug.cgi?id=4155
Summary Temporary WebCore SPI necessary for SVG testing tools
Eric Seidel (no email)
Reported 2005-07-26 23:47:41 PDT
Temporary WebCore SPI necessary for SVG testing tools. This adds a few Obj-C wrappers around kcanvas, ksvg2 to allow writing Obj-C based testing tools which link against WebCore+SVG. This SPI is only temporary, and will only exist in WebCore+SVG until SVG is well enough integrated that rendering can happen using the normal WebKit draw methods, and dom manipulation can happen through the WebKit exposed Obj-C dom classes.
Attachments
SPI additions necessary for builidng SVG test tools (81.26 KB, patch)
2005-07-27 17:29 PDT, Eric Seidel (no email)
sullivan: review-
Improved version: (more comments, spelling correction) (82.86 KB, patch)
2005-07-29 12:00 PDT, Eric Seidel (no email)
sullivan: review+
Eric Seidel (no email)
Comment 1 2005-07-27 17:29:56 PDT
Created attachment 3126 [details] SPI additions necessary for builidng SVG test tools
John Sullivan
Comment 2 2005-07-29 11:40:36 PDT
Comment on attachment 3126 [details] SPI additions necessary for builidng SVG test tools Given that this is all temporary code to be used for testing, I didn't review it really carefully. I did point out a couple of small things worth correcting on IRC, so I'm giving this a review- and will give the slightly improved version a review+.
Eric Seidel (no email)
Comment 3 2005-07-29 12:00:40 PDT
Created attachment 3148 [details] Improved version: (more comments, spelling correction) I added a bunch of comments, mostly trying to explain some of the hacks contained in this SPI. None of this code was ever really meant to "see the light of day" but I feel that releasing it now will help un-block other aspects of SVG development, and when things like Obj-C DOM interface generation, or the WC-DOM/KDOM merge are complete all of this code will be deleted.
John Sullivan
Comment 4 2005-07-29 12:05:24 PDT
Comment on attachment 3148 [details] Improved version: (more comments, spelling correction) Checking this in as-is seems fine, given its temporary nature.
Note You need to log in before you can comment on or make changes to this bug.