Summary: | Remove obsolete custom bindings file | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, eric, yaar | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | OS X 10.5 | ||||||||
Attachments: |
|
Description
Adam Barth
2010-04-27 15:10:13 PDT
Created attachment 54459 [details]
Patch
Comment on attachment 54459 [details]
Patch
There are more, but OK.
Comment on attachment 54459 [details]
Patch
More build systems or more files to remove?
Probably both. :) But I meant more files to remove. grep -L WebCore bindings/js Shows some header files needing removal: bindings/js/JSCSSStyleDeclarationCustom.h bindings/js/JSHistoryCustom.h bindings/js/JSHTMLAppletElementCustom.h bindings/js/JSHTMLEmbedElementCustom.h bindings/js/JSHTMLInputElementCustom.h bindings/js/JSHTMLObjectElementCustom.h bindings/js/JSLocationCustom.h bindings/js/JSStorageCustom.h grep -r -L "::" bindings/js/*.cpp Shows some likely candidates as well: bindings/js/JSAbstractWorkerCustom.cpp bindings/js/JSCDATASectionCustom.cpp bindings/js/JSCSSValueCustom.cpp bindings/js/JSDocumentFragmentCustom.cpp bindings/js/JSEventSourceCustom.cpp bindings/js/JSPluginElementFunctions.cpp bindings/js/JSTextCustom.cpp > bindings/js/JSDocumentFragmentCustom.cpp
Of the cpp files, only this one looked ready to remove.
Created attachment 54466 [details]
Patch
Comment on attachment 54466 [details]
Patch
OK. We can go back and remove the headers later. :)
Comment on attachment 54466 [details] Patch Clearing flags on attachment: 54466 Committed r58379: <http://trac.webkit.org/changeset/58379> All reviewed patches have been landed. Closing bug. |