When using the default pointer-events value, polylines should react to hit testing if filled with paint. In ToT this fails atm. Will append a testcase soon.
Created attachment 10846 [details] First attempt Fix for bug 11096 :) Cheers, Rob.
Comment on attachment 10846 [details] First attempt Looks good. But you need a better comment. Here is a suggestion: // CGPathContainsPoint returns false for non-closed paths, as a work-around, we copy and close the path first. Radar 4758998 asks for a better CG API to use We could also consider only ever copying the path if CGPathContainsPoint returns false. I'm just not sure how expensive CGPathContainsPoint is relative to a malloc.
I was just discussing this with rob again in the channel. This code is run on mousemove (if you're mousing over an SVG, so the malloc could be a significant problem. Another optimization could be to first check the bbox, only if it's in the bbox check CGPathContainsPoint, then only if that returned false, do we copy the path close it, and check again. Not sure all that optimization is necessary. We'd have to shark it.
Comment on attachment 10846 [details] First attempt Marking this r- until a solution is found, so this won't show up in the commit queue.
This is fixed on TOT. I fixed this when fixing an earlier related bug (in way which also causes copy/mallocs, unfortunately. :( *** This bug has been marked as a duplicate of 11931 ***