Refactor SMIL animation value updates

Refactor SMIL animation value updates

Push updating of the animation value into SVGElement. This resembles the
Web Animations code-path to some degree and maybe we can make them even
more similar eventually. This is the first CL in a series that will
remove knowledge of <use>/shadow trees from the SMIL animation code.

BUG=640676
Committed: https://crrev.com/451966ed6788fba2dd2526db15f859c7013b6c13
Cr-Commit-Position: refs/heads/master@{#414366}

[fs, 2016-08-24 16:46:32]

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

[commit-bot: I haz the power, 2016-08-24 16:47:16]

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

[fs, 2016-08-24 17:44:35]

Description was changed from

==========
Refactor SMIL animation value updates

Push updating of the animation value into SVGElement. This resembles the
Web Animations code-path to some degree and maybe we can make them even
more similar eventually. This is the first CL in a series that will
remove knowledge of <use>/shadow trees from the SMIL animation code.

BUG=Maybe later
==========

to

==========
Refactor SMIL animation value updates

Push updating of the animation value into SVGElement. This resembles the
Web Animations code-path to some degree and maybe we can make them even
more similar eventually. This is the first CL in a series that will
remove knowledge of <use>/shadow trees from the SMIL animation code.

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

[fs, 2016-08-24 17:46:10]

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

[fs, 2016-08-24 17:46:11]

Maybe I should've left the DCHECK_EQ(shouldApply, ApplyCSSAnimation) change in
SVGAnimateElement.cpp out of this CL, but...

[commit-bot: I haz the power, 2016-08-24 19:04:16]

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

[commit-bot: I haz the power, 2016-08-24 19:04:17]

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...)

[pdr., 2016-08-24 19:59:13]

On 2016/08/24 at 17:46:11, fs wrote:
> Maybe I should've left the DCHECK_EQ(shouldApply, ApplyCSSAnimation) change in
SVGAnimateElement.cpp out of this CL, but...

LGTM. Straightforward patch and your goal is an awesome one.

[fs, 2016-08-25 08:21:34]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-25 08:21: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-25 09:06:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 09:08:36]

Description was changed from

==========
Refactor SMIL animation value updates

Push updating of the animation value into SVGElement. This resembles the
Web Animations code-path to some degree and maybe we can make them even
more similar eventually. This is the first CL in a series that will
remove knowledge of <use>/shadow trees from the SMIL animation code.

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

to

==========
Refactor SMIL animation value updates

Push updating of the animation value into SVGElement. This resembles the
Web Animations code-path to some degree and maybe we can make them even
more similar eventually. This is the first CL in a series that will
remove knowledge of <use>/shadow trees from the SMIL animation code.

BUG=640676

Committed: https://crrev.com/451966ed6788fba2dd2526db15f859c7013b6c13
Cr-Commit-Position: refs/heads/master@{#414366}
==========

[commit-bot: I haz the power, 2016-08-25 09:08:37]

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