...
Created attachment 294474 [details] Patch
Comment on attachment 294474 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=294474&action=review > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:1795 > + die "Only DOMJIT=Getter is supported for attribute" unless $codeGenerator->ExtendedAttributeContains($attribute->extendedAttributes->{DOMJIT}, "Getter"); This would read slightly better if it said "Only DOMJIT=Getter is supported for attributes". Also, I prefer using the assert() function rather than die, since it gives you a nice stack trace. > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:3407 > + die "Only DOMJIT=Getter is supported for attribute" unless $codeGenerator->ExtendedAttributeContains($attribute->extendedAttributes->{DOMJIT}, "Getter"); Same.
Comment on attachment 294474 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=294474&action=review Thanks! >> Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:1795 >> + die "Only DOMJIT=Getter is supported for attribute" unless $codeGenerator->ExtendedAttributeContains($attribute->extendedAttributes->{DOMJIT}, "Getter"); > > This would read slightly better if it said "Only DOMJIT=Getter is supported for attributes". Also, I prefer using the assert() function rather than die, since it gives you a nice stack trace. Oh, nice! Use assert and the above phrase. >> Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:3407 >> + die "Only DOMJIT=Getter is supported for attribute" unless $codeGenerator->ExtendedAttributeContains($attribute->extendedAttributes->{DOMJIT}, "Getter"); > > Same. Fixed.
Committed r208582: <http://trac.webkit.org/changeset/208582>