this is part of a series of patches splitting down #16401 under an agreement suggested by david. gdom.h is a convenience header file that, by rights, should be auto-generated... but isn't yet. entertaining as it is to force people to #include over 350 header files, they might not be amused. hence the purpose of gdom.h. like they say on braniac "we do these things... so they don't have to..."
Created attachment 33058 [details] gdom.h developer convenience header should be auto-generated, but isn't. auto-generation can be considered a luxury for another time.
Comment on attachment 33058 [details] gdom.h developer convenience header Since I assume this is API for you all, you're welcome to name it however you like. It does need a license header though. r- for the lack of license, otherwise it looks fine. You should also fix your name from "lkcl" to your full name.
(In reply to comment #2) > (From update of attachment 33058 [details]) > Since I assume this is API for you all, you're welcome to name it however you > like. It does need a license header though. ahh, yes - whoops, i created it through grep "DerivedSources/gdom*.h" gdom | sort | uniq :) > r- for the lack of license, > otherwise it looks fine. > > You should also fix your name from "lkcl" to your full name. yes. sorry. that's now done across all 15 patches but wasn't at the time.
Created attachment 34288 [details] adds copyright notice and license
Comment on attachment 34288 [details] adds copyright notice and license Looks fine.
Comment on attachment 34288 [details] adds copyright notice and license commit-queue can't land this because the ChangeLog entry isn't at the top of the file.
Closing out GDOM bugs. Someone else will need to take these on later. See my email to webkit-dev on the subject.