Bug 12568 - WebKit fails mask-intro-01f from the W3c
Summary: WebKit fails mask-intro-01f from the W3c
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Eric Seidel (no email)
URL: http://www.w3.org/Graphics/SVG/Test/2...
Keywords: NeedsReduction
: 6409 (view as bug list)
Depends on:
Reported: 2007-02-04 03:19 PST by Eric Seidel (no email)
Modified: 2007-06-01 14:24 PDT (History)
1 user (show)

See Also:

partial fix (breaks one other mask test) (11.05 KB, patch)
2007-06-01 05:21 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff
fix and updated test cases (89.80 KB, patch)
2007-06-01 13:57 PDT, Eric Seidel (no email)
oliver: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2007-02-04 03:19:15 PST
WebKit fails mask-intro-01f from the W3c

This will need a reduction.
Comment 1 Eric Seidel (no email) 2007-06-01 05:21:28 PDT
Created attachment 14825 [details]
partial fix (breaks one other mask test)
Comment 2 Eric Seidel (no email) 2007-06-01 05:22:23 PDT
This was simply due to our lack of support of maskUnits and maskContentUnits.  The attached patch fixes that (sorta).  Unfortunately I regressed one other mask test with this patch, so I'll need to fix that before it can be landed.
Comment 3 Eric Seidel (no email) 2007-06-01 13:57:29 PDT
*** Bug 6409 has been marked as a duplicate of this bug. ***
Comment 4 Eric Seidel (no email) 2007-06-01 13:57:52 PDT
Created attachment 14831 [details]
fix and updated test cases
Comment 5 Oliver Hunt 2007-06-01 14:19:12 PDT
Comment on attachment 14831 [details]
fix and updated test cases

Comment 6 Eric Seidel (no email) 2007-06-01 14:24:22 PDT
Landed on the feature-branch as r21944.  This one is slightly more risky to move to trunk (as it makes it more common to request huge buffers now that we correctly support maskContents).  However, I fixed a few exception crashes as a result of the more-common huge-buffer allocation, so this might actually include a p1 fix in it and need to be moved (at least in part) to TOT.