Same work as bug 50316, but for SVGAnimatedNumber.
Erm for SVGAnimatedString/SVGAnimatedEnumeration, not SVGAnimatedNumber (copy-paste error ;-)
Created attachment 75371 [details]
Created attachment 75373 [details]
Oops, uploaded the wrong version, ignore the first patch.
Attachment 75371 [details] did not build on qt:
Build output: http://queues.webkit.org/results/6799009
Ran check-webkit-style locally as the style bot doesn't post results :((
Fixed two tab usages, and two include order problems.
Comment on attachment 75373 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=75373&action=review
It would be slightly nicer to do the un-indenting separately.
> + // This defines a non-virtual "String& target() const" method before, that would clash with "virtual String target() const"
> + // in Element. That's why it's now named "String& svgTarget() const", to avoid the clash. The CodeGenerators take care
The first sentence here seems a little garbled. And I think you mean "declares" instead of "defines".
Attachment 75371 [details] did not build on win:
Build output: http://queues.webkit.org/results/6776008
Landed in r73130.