Bug 4332

Summary: Fix WebCore+SVG compile breakage from 4313
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: SVGAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P4    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
This patch fixes most of the issues, but not all.... yet.
none
Should be a complete fix.
none
Final patch.
none
Adding call -> callFunction() change per darin's suggestion. darin: review+

Description Eric Seidel (no email) 2005-08-08 01:40:30 PDT
Now that http://bugzilla.opendarwin.org/show_bug.cgi?id=4313 is landed, kdom and friends are broken.

This needs to be fixed.
Comment 1 Eric Seidel (no email) 2005-08-08 01:41:28 PDT
Created attachment 3276 [details]
This patch fixes most of the issues, but not all.... yet.
Comment 2 Eric Seidel (no email) 2005-08-08 02:22:37 PDT
Created attachment 3277 [details]
Should be a complete fix.

I'm off to bed.  I'll run all the layout tests in the morning and commit.
Comment 3 Eric Seidel (no email) 2005-08-08 11:24:24 PDT
Created attachment 3281 [details]
Final patch.
Comment 4 Eric Seidel (no email) 2005-08-08 11:27:01 PDT
Comment on attachment 3281 [details]
Final patch.

Fixes WebCore+SVG to build again.  This patch shouldn't really need review,
since it's a build-fix, only affects kdom/ksvg2, and is doing the exact same
thing as darin's 4313.	However you now know about it if you care to comment. 
Unless anyone objects, I'll commit it later on.
Comment 5 Darin Adler 2005-08-08 11:38:53 PDT
Comment on attachment 3281 [details]
Final patch.

The call functions also have to be updated to use the new name, callAsFunction.
Comment 6 Eric Seidel (no email) 2005-08-08 13:12:09 PDT
Created attachment 3283 [details]
Adding call -> callFunction() change per darin's suggestion.
Comment 7 Eric Seidel (no email) 2005-08-08 13:13:17 PDT
Comment on attachment 3283 [details]
Adding call -> callFunction() change per darin's suggestion.

Again, I'll wait a bit to see if anyone has further comments, then will commit.
 All test SVG layout tests pass.
Comment 8 Darin Adler 2005-08-08 13:23:37 PDT
Comment on attachment 3283 [details]
Adding call -> callFunction() change per darin's suggestion.

Looks fine -- best if there was a way to test this, tho; Eric says we can't
test because of the JavaScript slot change.