Bug 6060 - gearflowers.svg renders incredibly slowly in Safari
Summary: gearflowers.svg renders incredibly slowly in Safari
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P4 Normal
Assignee: Nobody
URL: http://me.in-berlin.de/~darwin/svg/in...
Keywords:
Depends on: 6052
Blocks:
  Show dependency tree / graph
 
Reported: 2005-12-12 23:58 PST by Eric Seidel (no email)
Modified: 2008-03-23 13:38 PDT (History)
1 user (show)

See Also:

Attachments
Profile of gearflowers.svg being rendered (973.99 KB, application/octet-stream)
2005-12-17 13:29 PST, Ladd Van Tol 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2005-12-12 23:58:37 PST 
gearflowers.svg renders incredibly slowly in Safari

(It also currently renders wrong due to <stop> style issues: 6052)
Comment 1 Ladd Van Tol 2005-12-17 13:29:51 PST 
Created attachment 5124 [details]
Profile of gearflowers.svg being rendered

Part of this is that it is a 521k file on a slow server. A local copy takes on
order of 3 seconds to render on a dual 2 GHz G5 here. 7.9% of the time is
consumed in CoreGraphic's getNextAxialShadingScanline, which may be somewhat
optimizable. Other gradient-related functions are taking up a significant chunk
of time, athough the only WebCore function is cgGradientCallback.

See attached shark profile.
Comment 2 Eric Seidel (no email) 2006-12-26 17:14:35 PST 
Well, it's now part of our newly-landed SVG page load test.

Part of this is also a CG performance bug in shadings, which I expect will be addressed in a future OS release.
Comment 3 Eric Seidel (no email) 2007-10-12 02:25:30 PDT 
I filed a Radar with CG while I was still at apple about exactly this issue.  CG Gradients are incredibly slow.  We might just replace them with our own gradient code if we can't get a better API from them (functional based gradients make things way way way slower than they need to be) or see the perf issues with the existing gradient API addressed.  It would be nice to see that radar number associated with this bug (or a new one filed).
Comment 4 Eric Seidel (no email) 2008-03-23 13:38:32 PDT 
It's definitely better in leopard.
Comment 5 Eric Seidel (no email) 2008-03-23 13:38:58 PDT 
I'm not sure this bug is super-useful anymore.