Don't add redundant references to animated target element

Don't add redundant references to animated target element

SVGSMILElement adds a reference to its target, and any instances of the
target should never be mutated, so the registration of reference to the
extended target element set in SVGAnimateElement::resetAnimatedType does
not add any value.

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

[fs, 2016-08-30 13:00:27]

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

[commit-bot: I haz the power, 2016-08-30 13:00:41]

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-30 14:25:48]

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

[commit-bot: I haz the power, 2016-08-30 15:00:59]

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

[commit-bot: I haz the power, 2016-08-30 15:01:00]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[fs, 2016-08-30 15:45:38]

(In case you are/were wondering - I was for a bit there - why this code was
originally added, it was from a time when lifetimes were a royal PITA to manage,
and not just difficult like now...)

[pdr., 2016-08-30 17:27:03]

On 2016/08/30 at 15:45:38, fs wrote:
> (In case you are/were wondering - I was for a bit there - why this code was
originally added, it was from a time when lifetimes were a royal PITA to manage,
and not just difficult like now...)

I think you a word there, but I know what you mean :)

This code is getting so much better, keep 'em coming! LGTM

[fs, 2016-08-30 17:31:03]

On 2016/08/30 at 17:27:03, pdr wrote:
> On 2016/08/30 at 15:45:38, fs wrote:
> > (In case you are/were wondering - I was for a bit there - why this code was
originally added, it was from a time when lifetimes were a royal PITA to manage,
and not just difficult like now...)
> 
> I think you a word there, but I know what you mean :)
> 
> This code is getting so much better, keep 'em coming! LGTM

The sky^H^H^Hmy spare time is the limit!

[fs, 2016-08-30 17:31:11]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-30 17:31:37]

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-30 19:26:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 19:30:36]

Description was changed from

==========
Don't add redundant references to animated target element

SVGSMILElement adds a reference to its target, and any instances of the
target should never be mutated, so the registration of reference to the
extended target element set in SVGAnimateElement::resetAnimatedType does
not add any value.

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

to

==========
Don't add redundant references to animated target element

SVGSMILElement adds a reference to its target, and any instances of the
target should never be mutated, so the registration of reference to the
extended target element set in SVGAnimateElement::resetAnimatedType does
not add any value.

BUG=640676

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

[commit-bot: I haz the power, 2016-08-30 19:30:37]

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