Unify animate motion transform update invalidations

Unify animate motion transform update invalidations

SVGAnimateMotionElement changes transforms in 3 locations but does not
notify the targets of transform updates in a unified way. For example,
target->setNeedsTransformUpdate is called when the transform changes but
the target is not marked for layout until applyResultsToTarget. This
patch unifies these invalidations to always occur when the tranform
is updated. The previous code dates back to http://wkbug.com/41249.

BUG=645667
Committed: https://crrev.com/174b75c4971ebe89bf055b824c9c5ef0d5a75aee
Cr-Commit-Position: refs/heads/master@{#438841}

[pdr., 2016-12-15 05:35:18]

The CQ bit was checked by pdr@chromium.org to run a CQ dry run

[pdr., 2016-12-15 05:35:23]

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

[commit-bot: I haz the power, 2016-12-15 05:35:42]

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-15 07:04:12]

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

[commit-bot: I haz the power, 2016-12-15 07:04:12]

Dry run: This issue passed the CQ dry run.

[fs, 2016-12-15 08:13:42]

Don't know if you had intended to publish this, but this looks like it would
work, so LGTM.

It would be preferable to invalidate all target elements (direct + instances) in
applyResultToTarget (and eventually that should be possible...), but hopefully
this is good enough transformation.

[pdr., 2016-12-15 16:08:39]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-12-15 16:09:04]

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-15 16:13:28]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481818119298460, "parent_rev":
"3242846a49559dd6871075f4952d1689f866e859", "commit_rev":
"62fc89201e347f27ba18c24452075f1722229f5f"}

[commit-bot: I haz the power, 2016-12-15 16:13:54]

Description was changed from

==========
Unify animate motion transform update invalidations

SVGAnimateMotionElement changes transforms in 3 locations but does not
notify the targets of transform updates in a unified way. For example,
target->setNeedsTransformUpdate is called when the transform changes but
the target is not marked for layout until applyResultsToTarget. This
patch unifies these invalidations to always occur when the tranform
is updated. The previous code dates back to http://wkbug.com/41249.

BUG=645667
==========

to

==========
Unify animate motion transform update invalidations

SVGAnimateMotionElement changes transforms in 3 locations but does not
notify the targets of transform updates in a unified way. For example,
target->setNeedsTransformUpdate is called when the transform changes but
the target is not marked for layout until applyResultsToTarget. This
patch unifies these invalidations to always occur when the tranform
is updated. The previous code dates back to http://wkbug.com/41249.

BUG=645667

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

[commit-bot: I haz the power, 2016-12-15 16:13:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 16:16:08]

Description was changed from

==========
Unify animate motion transform update invalidations

SVGAnimateMotionElement changes transforms in 3 locations but does not
notify the targets of transform updates in a unified way. For example,
target->setNeedsTransformUpdate is called when the transform changes but
the target is not marked for layout until applyResultsToTarget. This
patch unifies these invalidations to always occur when the tranform
is updated. The previous code dates back to http://wkbug.com/41249.

BUG=645667

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

to

==========
Unify animate motion transform update invalidations

SVGAnimateMotionElement changes transforms in 3 locations but does not
notify the targets of transform updates in a unified way. For example,
target->setNeedsTransformUpdate is called when the transform changes but
the target is not marked for layout until applyResultsToTarget. This
patch unifies these invalidations to always occur when the tranform
is updated. The previous code dates back to http://wkbug.com/41249.

BUG=645667

Committed: https://crrev.com/174b75c4971ebe89bf055b824c9c5ef0d5a75aee
Cr-Commit-Position: refs/heads/master@{#438841}
==========

[commit-bot: I haz the power, 2016-12-15 16:16:09]

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