Hoist updateAnimation() calls from SVGSMILElement::progress

Hoist updateAnimation() calls from SVGSMILElement::progress

The updateAnimation() calls in SVGSMILElement::progress() corresponds
to the timed element contributing to the animation "sandwich", so it's
trivial to hoist it out into the caller. Well there, the |resultElement|
logic can be replaced with a vector onto contributing timed elements are
appended. When all contributing timed elements are collected the value
is computed.

BUG=641437
Committed: https://crrev.com/c4f7094a1f713541aab18286d664f8dae1826eba
Cr-Commit-Position: refs/heads/master@{#414937}

[fs, 2016-08-26 16:22:59]

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

[commit-bot: I haz the power, 2016-08-26 16:23:53]

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-26 18:04:09]

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

[commit-bot: I haz the power, 2016-08-26 18:04:10]

Dry run: This issue passed the CQ dry run.

[fs, 2016-08-26 18:48:05]

Description was changed from

==========
Hoist updateAnimation() calls from SVGSMILElement::progress
==========

to

==========
Hoist updateAnimation() calls from SVGSMILElement::progress

The updateAnimation() calls in SVGSMILElement::progress() corresponds
to the timed element contributing to the animation "sandwich", so it's
trivial to hoist it out into the caller. Well there, the |resultElement|
logic can be replaced with a vector onto contributing timed elements are
appended. When all contributing timed elements are collected the value
is computed.

BUG=641437
==========

[fs, 2016-08-26 18:49:29]

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

[fs, 2016-08-26 18:49:30]

The latest invention from the Crazy Clown's SMIL(e) Factory.

[pdr., 2016-08-26 22:04:16]

On 2016/08/26 at 18:49:30, fs wrote:
> The latest invention from the Crazy Clown's SMIL(e) Factory.

I image searched for "crazy clown smile" and this patch is significantly better.

LGTM

[fs, 2016-08-27 15:39:21]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-27 15:39:32]

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-27 15:42:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-27 15:44:46]

Description was changed from

==========
Hoist updateAnimation() calls from SVGSMILElement::progress

The updateAnimation() calls in SVGSMILElement::progress() corresponds
to the timed element contributing to the animation "sandwich", so it's
trivial to hoist it out into the caller. Well there, the |resultElement|
logic can be replaced with a vector onto contributing timed elements are
appended. When all contributing timed elements are collected the value
is computed.

BUG=641437
==========

to

==========
Hoist updateAnimation() calls from SVGSMILElement::progress

The updateAnimation() calls in SVGSMILElement::progress() corresponds
to the timed element contributing to the animation "sandwich", so it's
trivial to hoist it out into the caller. Well there, the |resultElement|
logic can be replaced with a vector onto contributing timed elements are
appended. When all contributing timed elements are collected the value
is computed.

BUG=641437

Committed: https://crrev.com/c4f7094a1f713541aab18286d664f8dae1826eba
Cr-Commit-Position: refs/heads/master@{#414937}
==========

[commit-bot: I haz the power, 2016-08-27 15:44:47]

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