Bug 11163 - Improve opacity performance by clipping to bbox before starting transparency layer
Summary: Improve opacity performance by clipping to bbox before starting transparency ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P4 Blocker
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-05 05:48 PDT by Eric Seidel (no email)
Modified: 2006-10-05 14:50 PDT (History)
0 users

See Also:

Attachments
Pathological opacity test case (22.39 KB, image/svg+xml)
2006-10-05 05:50 PDT, Eric Seidel (no email) 		no flags Details
improve opacity performance (5.33 KB, patch)
2006-10-05 05:51 PDT, Eric Seidel (no email) 		mjs: review+
Details | Formatted Diff | Diff
Improved test case w/ timing information (click while beach-balling) (22.54 KB, image/svg+xml)
2006-10-05 06:02 PDT, Eric Seidel (no email) 		no flags Details
Show Obsolete (1) 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) 2006-10-05 05:48:14 PDT 
Improve opacity performance by clipping to bbox before starting transparency layer

Calling beginTransparencyLayer() is slow, because it results in creating a new context the size of the current clip region.  We can improve this performance by reducing the size of the current clip region to the smallest required size.  This patch improves opacity performance in SVG.
Comment 1 Eric Seidel (no email) 2006-10-05 05:50:27 PDT 
Created attachment 10921 [details]
Pathological opacity test case
Comment 2 Eric Seidel (no email) 2006-10-05 05:51:06 PDT 
Created attachment 10922 [details]
improve opacity performance
Comment 3 Maciej Stachowiak 2006-10-05 05:57:41 PDT 
Comment on attachment 10922 [details]
improve opacity performance

r=me
Comment 4 Eric Seidel (no email) 2006-10-05 06:02:24 PDT 
Created attachment 10923 [details]
Improved test case w/ timing information (click while beach-balling)

Pre-patch numbers are around 8000 on my machine.  Post-patch, around 1500.
Comment 5 Mark Rowe (bdash) 2006-10-05 14:50:03 PDT 
This patch was landed in r16802.