Remove SVGAnimateElement::hasValidAttributeType

Remove SVGAnimateElement::hasValidAttributeType

Incorporate said method into its only caller - hasValidTarget - and
override that one in SVGAnimateTransformElement instead.
Also fold animatedPropertyType() into hasValidTarget and get rid of
hasInvalidCSSAttributeType() in favor of "naked" access to the
underlying boolean flag.

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

[fs, 2016-12-23 12:15:09]

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

[commit-bot: I haz the power, 2016-12-23 12:15:18]

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-12-23 14:36:42]

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

[commit-bot: I haz the power, 2016-12-23 14:36:42]

Dry run: This issue passed the CQ dry run.

[fs, 2016-12-23 14:40:09]

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

[Stephen Chennney, 2016-12-23 20:02:14]

lgtm

[fs, 2016-12-26 15:51:25]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-12-26 15:51:36]

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-12-26 17:17:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482767485997010, "parent_rev":
"b8e87581eed46bf42b1a02eb8640a9e6f14682e5", "commit_rev":
"9671077f3c6c8f0b7c1beeb08061664118502f59"}

[commit-bot: I haz the power, 2016-12-26 17:18:15]

Description was changed from

==========
Remove SVGAnimateElement::hasValidAttributeType

Incorporate said method into its only caller - hasValidTarget - and
override that one in SVGAnimateTransformElement instead.
Also fold animatedPropertyType() into hasValidTarget and get rid of
hasInvalidCSSAttributeType() in favor of "naked" access to the
underlying boolean flag.

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

to

==========
Remove SVGAnimateElement::hasValidAttributeType

Incorporate said method into its only caller - hasValidTarget - and
override that one in SVGAnimateTransformElement instead.
Also fold animatedPropertyType() into hasValidTarget and get rid of
hasInvalidCSSAttributeType() in favor of "naked" access to the
underlying boolean flag.

BUG=641437

Review-Url: https://codereview.chromium.org/2599853003
==========

[commit-bot: I haz the power, 2016-12-26 17:18:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-02 15:45:04]

Description was changed from

==========
Remove SVGAnimateElement::hasValidAttributeType

Incorporate said method into its only caller - hasValidTarget - and
override that one in SVGAnimateTransformElement instead.
Also fold animatedPropertyType() into hasValidTarget and get rid of
hasInvalidCSSAttributeType() in favor of "naked" access to the
underlying boolean flag.

BUG=641437

Review-Url: https://codereview.chromium.org/2599853003
==========

to

==========
Remove SVGAnimateElement::hasValidAttributeType

Incorporate said method into its only caller - hasValidTarget - and
override that one in SVGAnimateTransformElement instead.
Also fold animatedPropertyType() into hasValidTarget and get rid of
hasInvalidCSSAttributeType() in favor of "naked" access to the
underlying boolean flag.

BUG=641437

Committed: https://crrev.com/5795f556552fe3d6bcd19d8854ee97d223c96cbf
Cr-Commit-Position: refs/heads/master@{#440716}
==========

[commit-bot: I haz the power, 2017-01-02 15:45:05]

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