Devirtualize animatedLocalTransform() and rename it for clarity

Devirtualize animatedLocalTransform() and rename it for clarity

This patch devirtualizes animatedLocalTransform and renames it
calculateAnimatedLocalTransform to emphasize that it is not cheap.

SVGGraphicsElement::animatedLocalTransform was virtual to support
SVGTextElement's special transform code. The text-specific
transform code is only in an infrequently-taken codepath
(style && style->hasTransform()) so we can actually improve performance
by devirtalizing and combining the two transform codepaths. The real
benefit is code simplification though.

Secondly, I've renamed this function to make it clear that it is not
a simple lookup.

BUG=429551
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184845

[pdr., 2014-11-02 23:37:56]

pdr@chromium.org changed reviewers:
+ fs@opera.com, schenney@chromium.org

[pdr., 2014-11-04 06:26:18]

pdr@chromium.org changed reviewers:
+ fmalita@chromium.org

[Stephen Chennney, 2014-11-04 19:00:26]

Looks fine to me. Took me a while to verify the code does the same thing. LGTM.

[pdr., 2014-11-04 20:48:14]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2014-11-04 20:48:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/693423002/20001

[commit-bot: I haz the power, 2014-11-05 00:12:24]

Committed patchset #2 (id:20001) as 184845