Fold svgAttributeChanged into parseAttribute for SVGAnimateElement

Fold svgAttributeChanged into parseAttribute for SVGAnimateElement

This separation does not really give much in terms of benefits. Since
none of the SMIL-related attributes are exposed through SVG DOM, the
former method is always called after the latter. Move the code from the
former to the latter. This even avoids the attribute value lookup, for a
tiny perf gain...

BUG=641437
Review-Url: https://codereview.chromium.org/2706243012
Cr-Commit-Position: refs/heads/master@{#452861}
Committed: https://chromium.googlesource.com/chromium/src/+/d7d7475a474a473efc0b9578305e663653df052f

[fs, 2017-02-24 13:28:20]

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

[commit-bot: I haz the power, 2017-02-24 13:28:37]

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

[fs, 2017-02-24 14:22:08]

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

[commit-bot: I haz the power, 2017-02-24 14:45:00]

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

[commit-bot: I haz the power, 2017-02-24 14:45:01]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-02-24 16:14:59]

On 2017/02/24 14:22:08, fs wrote:

lgtm

[fs, 2017-02-24 17:24:45]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2017-02-24 17:25:11]

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, 2017-02-24 17:35:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487957085879160, "parent_rev":
"499496f9c32df032e54361ad474497e65e405556", "commit_rev":
"d7d7475a474a473efc0b9578305e663653df052f"}

[commit-bot: I haz the power, 2017-02-24 17:36:04]

Description was changed from

==========
Fold svgAttributeChanged into parseAttribute for SVGAnimateElement

This separation does not really give much in terms of benefits. Since
none of the SMIL-related attributes are exposed through SVG DOM, the
former method is always called after the latter. Move the code from the
former to the latter. This even avoids the attribute value lookup, for a
tiny perf gain...

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

to

==========
Fold svgAttributeChanged into parseAttribute for SVGAnimateElement

This separation does not really give much in terms of benefits. Since
none of the SMIL-related attributes are exposed through SVG DOM, the
former method is always called after the latter. Move the code from the
former to the latter. This even avoids the attribute value lookup, for a
tiny perf gain...

BUG=641437

Review-Url: https://codereview.chromium.org/2706243012
Cr-Commit-Position: refs/heads/master@{#452861}
Committed:
https://chromium.googlesource.com/chromium/src/+/d7d7475a474a473efc0b9578305e...
==========

[commit-bot: I haz the power, 2017-02-24 17:36:05]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d7d7475a474a473efc0b9578305e...