commit 19f7c54a725c49bbc2bc683d1069c2217f302cb9
author fs <fs@opera.com> Mon Oct 03 19:15:35 2016
committer Commit bot <commit-bot@chromium.org> Mon Oct 03 19:17:52 2016
tree d89e4d411ce943d9c470e37d9a44ae036e8a5455
parent 4b321f75b1e568814c230362ad87b44594ed2c25

Hoist target element null-checks out of SVGAnimateElement::calculate*

The calculateFromAndToValues, calculateFromAndByValues and
calculateDistance share a common entrypoint (startedActiveInterval), so
we can tighten this code-path a bit checking for a target element up
front. (No target element makes for pretty useless animations anyway...)
calculateAnimatedValue gets similar treatment in its updateAnimation
(sole) entrypoint.

Also refill comments to 80 columns in SVGAnimateMotionElement.cpp.

BUG=641437

Review-Url: https://codereview.chromium.org/2386013002
Cr-Commit-Position: refs/heads/master@{#422484}