Some of this, some of that. Patch forthcoming.
Created attachment 10608 [details] patch - Whitespace cleanup for IDLParser.pm - Add conditional #ifdefs (ie. #ifdef SVG_SUPPORT) to auto-generated implementation files. - Split DOMException, DOMRangeException, DOMXPathException, DOMEventsException and DOMAbstractView into their own files. - Remove private method [DOMRGBColor _color] from DOMPrivate.h asit is not used and a public method is available now. - Remove internal method [DOMDocument _ownerElement] from DOMInternal.h as it not used. - Auto-generate method isContentEditable for DOMNode. - General cleanup of bindings code including removing unneeded #imports and whitespace cleanup.
Comment on attachment 10608 [details] patch +#import <WebCore/DOMDOMImplementation.h> THis was in the original order because once in WebKit is will be: #import <WebCore/DOMImplementation.h> So the order in WeCore is wrong but it will be right once in WebKit. DOMObject.h needs to include DOMException.h since you removed the enum from there. isContentEditable should be for ObjC only. This naming schee is only common in ObjC (with the "is" prefix for boolean types.) Other languages should just be "bool contentEditible". I am fine keeping isContentEditable as a hand implemented method.
Created attachment 10614 [details] patch with Tim's comments addressed
Created attachment 10615 [details] patch with Tim's comments addressed
Comment on attachment 10615 [details] patch with Tim's comments addressed The #import <WebCore/DOMDOMImplementation.h> should not be moved. DOMObject.h still needs to import DOMException.h since the exceptions are being removed from that header. We can't move them unless someone that includes DOMObject.h still gets the exception enums.
r=me otherwise
Created attachment 10633 [details] patch with all of Tim's comments addressed
Landed in r16438.