CodeGeneratorV8.pm has its own copies of WK_ucfirst and WK_lcfirst. It should just call the ones in CodeGenerator.pm instead.
Created attachment 39399 [details] Patch v1 I note that the WK_lcfirst in CodeGenerator.pm is different from the one in CodeGeneratorV8.pm. The only case that CodeGeneratorV8 calls WK_lcfirst for is the "URL" attribute on Document (and a couple of other interfaces), so there should be no change in output. I inspected the output of running generate-bindings.pl on Document visually, but didn't build anything to test (I guess I'd need to check out Chromium to do that).
Comment on attachment 39399 [details] Patch v1 You're a brave man. :) I guess we'll just roll this out if you break the Chromium canary.
This is a good clean-up. cam++.
Comment on attachment 39399 [details] Patch v1 Clearing flags on attachment: 39399 Committed r48292: <http://trac.webkit.org/changeset/48292>
All reviewed patches have been landed. Closing bug.