6564 2006-01-15 11:22:10 -0800 SVGPaintServer system needs to be redesigned to fill & stroke simultaneously 2011-05-18 10:10:55 -0700 1 1 1 Unclassified WebKit SVG 420+ Mac OS X 10.4 RESOLVED WONTFIX P4 Normal --- 6553 1 a webkit-unassigned ian krit oldest_to_newest 28522 0 a 2006-01-15 11:22:10 -0800 KCanvas needs to be redesigned to fill & stroke at once This is a continuation of 6553. Items remaining: 1. Update PaintServers to not paint directly, but rather have an apply(STROKE | FILL) method which applies their stoke/fill properties to the current context. 2. Change RenderPathQuartz to call CGContextDrawPath, and thus both stroke and fill a path in a single call (large perf. gain). 30202 1 eric 2006-01-26 16:13:57 -0800 This could be a big performance win on large SVGs. 39734 2 eric 2006-12-26 17:23:25 -0800 This is nearly possible today. Just a few adjustments would be necessary. Here's some pseudo code: fill = fillPaintServer() stroke = strokePaintServer() if (fill == stroke) { fill->setup() fill->renderPath(FILL_MODE | STROKE_MODE) fill->teardown() } else { fill->setup() fill->renderPath(FILL_MODE) fill->teardown() stroke->setup() stroke->renderPath(STROKE_MODE) stroke->teardown() } I'm not certain how huge of a performance win this would be, but now that we have an SVG PLT we can check. ;) 405796 3 krit 2011-05-18 10:10:55 -0700 I doubt that it makes sense to implement it. We need to be able to apply different transforms to the context for fill or stroke (because of gradients, patterns, non-scaling-stroke). And IIRC this won't be able with drawing fill and stroke at once for CG. Also just CG could make use of this. Because of the cost to implement it and the fact that just CG could make use of it. I mark this as WONTFIX. Feel free to reopen the bug if you want to work on that.