WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
8983
Autogenerate another 22 HTML classes
https://bugs.webkit.org/show_bug.cgi?id=8983
Summary
Autogenerate another 22 HTML classes
Anders Carlsson
Reported
2006-05-18 15:39:48 PDT
Patch coming up
Attachments
Patch
(143.11 KB, patch)
2006-05-18 15:45 PDT
,
Anders Carlsson
eric
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Anders Carlsson
Comment 1
2006-05-18 15:45:44 PDT
Created
attachment 8398
[details]
Patch I've been very careful not to remove any existing attributes/functions while porting.
Eric Seidel (no email)
Comment 2
2006-05-18 16:14:29 PDT
Comment on
attachment 8398
[details]
Patch In general this looks great. We shoudl make sure to land a test for every place in which we differ from the spec. In autogenerating other classes I found that the dom 2 html spec changed the type of several attributes, which might be what you're noticing in these as well. Second, you also should fix the style Document *doc instead of Document* doc, in HTMLBodyElement.cpp With those additional test cases to cover the places where we differ from the spec, you're good to land.
Anders Carlsson
Comment 3
2006-05-19 03:50:37 PDT
I did not want to change too much of the implementation logic, so I filed
http://bugzilla.opendarwin.org/show_bug.cgi?id=8992
to keep track of the three attributes where we differ.
Darin Adler
Comment 4
2006-05-19 08:48:36 PDT
Anders landed this. Version 14483. Anders, it looks like the XMLHttpRequest layout tests are now failing. We need to fix that. And perhaps the Windows build as well.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug