Need to fix the support for static conditional overloaded functions in JSC. For all other frameworks its working. When overloaded functions are conditional, there are two local static functions are created. The bindings generator script pushes this "static" keyword first and then pushes "#if {conditional}" later which causes incorrect bindings being generated.
Created attachment 118350 [details] Patch
Comment on attachment 118350 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=118350&action=review > Source/WebCore/bindings/scripts/test/CPP/WebDOMTestObj.cpp:909 > +#if ENABLE(Condition1) It would be nice to have a blank line after this line, but that's really a minor nit. :)
(In reply to comment #2) > (From update of attachment 118350 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=118350&action=review > > > Source/WebCore/bindings/scripts/test/CPP/WebDOMTestObj.cpp:909 > > +#if ENABLE(Condition1) > > It would be nice to have a blank line after this line, but that's really a minor nit. :) Thanks Adam for quick review :)
Comment on attachment 118350 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=118350&action=review This does not do V8. I'm fine without V8 support in this patch. I can add it in another patch. > Source/WebCore/bindings/scripts/test/TestObj.idl:171 > + static [Conditional=Condition1] void overloadedMethod1(in long arg); > + static [Conditional=Condition1] void overloadedMethod1(in DOMString type); Can this be called overloadedClassMethod instead? > Source/WebCore/bindings/scripts/test/V8/V8TestObj.cpp:1288 > > +static v8::Handle<v8::Value> overloadedMethod11Callback(const v8::Arguments& args) This needs #if ENABLE(Condition1) too.
Comment on attachment 118350 [details] Patch Clearing flags on attachment: 118350 Committed r102351: <http://trac.webkit.org/changeset/102351>
All reviewed patches have been landed. Closing bug.