Fix synchronization of SVGAnimatedAngle (<marker orient>)

Fix synchronization of SVGAnimatedAngle (<marker orient>)

Since SVGAnimatedAngle also wraps the SVGAnimatedEnumeration for the
enumeration representation of the 'orient' attribute, and both of them
synchronize to said attribute, we need to override the synchronization
methods to take the synchronization status of them both into account
(as opposed to previously where only the SVGAnimatedAngle itself was
considered.) Rewrite the existing synchronizeAttribute() implementation
to just delegate rather than do the actual work itself.

Also change reference from SVGMarkerElement to just SVGElement and
include the specific header - SVGAngle.h rather than SVGAngleTearOff.h.

BUG=589808
Committed: https://crrev.com/8b5d6fa330a1ffb342db4a72a11982b8fa995722
Cr-Commit-Position: refs/heads/master@{#377875}

[fs, 2016-02-25 17:08:24]

Description was changed from

==========
Fix synchronization of SVGAnimatedAngle (<marker orient>)

Since SVGAnimatedAngle also wraps the SVGAnimatedEnumeration for the
enumeration representation of the 'orient' attribute, and both of them
synchronize to said attribute, we need to override the synchronization
methods to take the synchronization status of them both into account
(as opposed to previously where only the SVGAnimatedAngle itself was
considered.)

Also change reference from SVGMarkerElement to just SVGElement and
include the specific header - SVGAngle.h rather than SVGAngleTearOff.h.

BUG=589808
==========

to

==========
Fix synchronization of SVGAnimatedAngle (<marker orient>)

Since SVGAnimatedAngle also wraps the SVGAnimatedEnumeration for the
enumeration representation of the 'orient' attribute, and both of them
synchronize to said attribute, we need to override the synchronization
methods to take the synchronization status of them both into account
(as opposed to previously where only the SVGAnimatedAngle itself was
considered.) Rewrite the existing synchronizeAttribute() implementation
to just delegate rather than do the actual work itself.

Also change reference from SVGMarkerElement to just SVGElement and
include the specific header - SVGAngle.h rather than SVGAngleTearOff.h.

BUG=589808
==========

[fs, 2016-02-26 11:30:14]

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

[davve, 2016-02-26 12:42:06]

Patch looks obviously correct. LGTM.

This part of the SVG DOM otoh... *sigh*

[fs, 2016-02-26 13:14:00]

On 2016/02/26 at 12:42:06, davve wrote:
> Patch looks obviously correct. LGTM.
> 
> This part of the SVG DOM otoh... *sigh*

I'll see your *sigh* and raise you a *facepalm*

[fs, 2016-02-26 13:14:25]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-02-26 13:14:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739533004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739533004/20001

[commit-bot: I haz the power, 2016-02-26 13:19:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-26 13:20:23]

Description was changed from

==========
Fix synchronization of SVGAnimatedAngle (<marker orient>)

Since SVGAnimatedAngle also wraps the SVGAnimatedEnumeration for the
enumeration representation of the 'orient' attribute, and both of them
synchronize to said attribute, we need to override the synchronization
methods to take the synchronization status of them both into account
(as opposed to previously where only the SVGAnimatedAngle itself was
considered.) Rewrite the existing synchronizeAttribute() implementation
to just delegate rather than do the actual work itself.

Also change reference from SVGMarkerElement to just SVGElement and
include the specific header - SVGAngle.h rather than SVGAngleTearOff.h.

BUG=589808
==========

to

==========
Fix synchronization of SVGAnimatedAngle (<marker orient>)

Since SVGAnimatedAngle also wraps the SVGAnimatedEnumeration for the
enumeration representation of the 'orient' attribute, and both of them
synchronize to said attribute, we need to override the synchronization
methods to take the synchronization status of them both into account
(as opposed to previously where only the SVGAnimatedAngle itself was
considered.) Rewrite the existing synchronizeAttribute() implementation
to just delegate rather than do the actual work itself.

Also change reference from SVGMarkerElement to just SVGElement and
include the specific header - SVGAngle.h rather than SVGAngleTearOff.h.

BUG=589808

Committed: https://crrev.com/8b5d6fa330a1ffb342db4a72a11982b8fa995722
Cr-Commit-Position: refs/heads/master@{#377875}
==========

[commit-bot: I haz the power, 2016-02-26 13:20:24]

Patchset 2 (id:??) landed as
https://crrev.com/8b5d6fa330a1ffb342db4a72a11982b8fa995722
Cr-Commit-Position: refs/heads/master@{#377875}