Simplify SVGAnimatedTypeAnimator interface

Simplify SVGAnimatedTypeAnimator interface

The two methods startAnimValAnimation and resetAnimValToBaseVal share
the same implementation, but are called in two different branches of the
same if-statement. Fold the two and add a new method
createAnimatedValue() to provide the animated value, to make this appear
as straight-forward as it is. Also fold the stopAnimValAnimation()
method since it's only used once, and because this makes the start/stop
action somewhat symmetric.
Rename the constructFromString(...) method to
createAnimatedValueFromString() to illustrate its relation to the newly
minted method and make it a bit more clear what it does.

BUG=640676
Committed: https://crrev.com/2e7202542f73e6a411d0c1808e2112a150254e3b
Cr-Commit-Position: refs/heads/master@{#415271}

[fs, 2016-08-28 19:21:50]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-28 19:22:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-28 20:33:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-28 20:33:19]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[fs, 2016-08-28 20:57:56]

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

[pdr., 2016-08-29 00:22:28]

On 2016/08/28 at 20:57:57, fs wrote:
> 

LGTM

[fs, 2016-08-29 08:13:09]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-29 08:13:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-29 08:13:26]

This CL has an open dependency (Issue 2287983002 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[fs, 2016-08-30 09:34:40]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-30 09:34:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 11:31:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 11:33:18]

Description was changed from

==========
Simplify SVGAnimatedTypeAnimator interface

The two methods startAnimValAnimation and resetAnimValToBaseVal share
the same implementation, but are called in two different branches of the
same if-statement. Fold the two and add a new method
createAnimatedValue() to provide the animated value, to make this appear
as straight-forward as it is. Also fold the stopAnimValAnimation()
method since it's only used once, and because this makes the start/stop
action somewhat symmetric.
Rename the constructFromString(...) method to
createAnimatedValueFromString() to illustrate its relation to the newly
minted method and make it a bit more clear what it does.

BUG=640676
==========

to

==========
Simplify SVGAnimatedTypeAnimator interface

The two methods startAnimValAnimation and resetAnimValToBaseVal share
the same implementation, but are called in two different branches of the
same if-statement. Fold the two and add a new method
createAnimatedValue() to provide the animated value, to make this appear
as straight-forward as it is. Also fold the stopAnimValAnimation()
method since it's only used once, and because this makes the start/stop
action somewhat symmetric.
Rename the constructFromString(...) method to
createAnimatedValueFromString() to illustrate its relation to the newly
minted method and make it a bit more clear what it does.

BUG=640676

Committed: https://crrev.com/2e7202542f73e6a411d0c1808e2112a150254e3b
Cr-Commit-Position: refs/heads/master@{#415271}
==========

[commit-bot: I haz the power, 2016-08-30 11:33:19]

Patchset 1 (id:??) landed as
https://crrev.com/2e7202542f73e6a411d0c1808e2112a150254e3b
Cr-Commit-Position: refs/heads/master@{#415271}