Summary: | SVGTextContentElement is missing all its DOM methods | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||||||||
Component: | SVG | Assignee: | Nikolas Zimmermann <zimmermann> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Enhancement | CC: | a.neumann, good4me, hyatt, justin.garcia, ml, oliver | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 420+ | ||||||||||||||
Hardware: | Mac | ||||||||||||||
OS: | OS X 10.4 | ||||||||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2007-01-15 18:11:41 PST
http://www.carto.net/neumann/webkitsvgbugs/text-dom-01-f.svg is testing all methods and parts of the properties of the SVGTextContentElement. I also uploaded it as a testcase. The testcase is in the W3C SVG testsuit format and will probably also be added to the testsuite. I will probably later provide individual test cases if you want. So far Apache Batik is the only SVG UA passing all the subtests. ASV3 passes most of the subtests. Created attachment 12483 [details] testing the interface SVGTextContentElement http://www.carto.net/neumann/webkitsvgbugs/text-dom-01-f.png shows how the correct rendering looks like in Batik Created attachment 12484 [details]
correct rendering of the test in Apache Batik.
Another example test: http://www.w3.org/Graphics/SVG/Test/20061213/htmlEmbedHarness/full-text-tselect-02-f.html I peaked at this again today. I expect this will be a couple hour hack for WildFox once he finds time. Eventually I'll spend the time to understand his fancy text-chunk system... but not tonight. *** Bug 15622 has been marked as a duplicate of this bug. *** Created attachment 17302 [details]
Initial patch
Created attachment 17303 [details]
Layout test results
Comment on attachment 17303 [details]
Layout test results
The layout test results look fine. They're of no use w/o the patch however, so just clearing the flag.
Comment on attachment 17302 [details]
Initial patch
This looks OK. We talked over IRC about a nicer design for the QueryWalker which used subclasses and specific variable names/methods instead of generic query variables. Possibly virtual methods or templates instead of a switch statement.
You're currently working out the nextPosition stuff with Harrison. When that's done we can review your final patch.
Created attachment 17304 [details]
Updated patch
A much better version of selectSubString() thanks to the help of Justin & Harrison.
Comment on attachment 17304 [details]
Updated patch
Looks fine.
Landed in r27848. |