|Summary:||SVGRenderStyle should store pre-modified resource URIs|
|Product:||WebKit||Reporter:||Eric Seidel (no email) <eric>|
|Version:||523.x (Safari 3)|
|OS:||OS X 10.4|
Description Eric Seidel (no email) 2007-10-08 17:50:31 PDT
SVGRenderStyle should store pre-modified resource URIs We have this code all over the place: getFilterById(document(), SVGURIReference::getTarget(style()->svgStyle()->filter())); And some other variations as well (manually doing what getTarget does). Instead, the SVGRenderStyle should store *just* the anchor as an id, or the full url. HTML does something similar. This could be done as a CSSValue, I think. That way we don't have to do lots of string manipulations every time we layout or paint SVG render objects (these string manipulations have shown up in samples in the past).
Comment 1 Rob Buis 2008-07-06 12:56:44 PDT
Created attachment 22114 [details] First attempt This moves the getTarget call to the style selector, unfortunately it changes a lot of testcase text results. Cheers, Rob.
Comment 2 Eric Seidel (no email) 2008-07-06 18:53:38 PDT
Comment on attachment 22114 [details] First attempt Will this break external references?
Comment 3 Rob Buis 2008-07-08 00:16:00 PDT
Hi Eric, (In reply to comment #2) > (From update of attachment 22114 [details] ) > Will this break external references? Do you mean external references for tref and use? This has not landed yet and these are not stored in (SVG)RenderStyle. Cheers, Rob.
Comment 4 Eric Seidel (no email) 2008-07-09 13:10:41 PDT
Comment on attachment 22114 [details] First attempt This is fine. We'll eventually have to come up with a solution for external references.