Revert of Add support for 'href' (w/o XLink NS) for various SVG elements

Revert of Add support for 'href' (w/o XLink NS) for various SVG elements (patchset #6 id:100001 of https://codereview.chromium.org/1681553002/ )

Reason for revert:
This CL seems to be breaking the compilation step of a lot of Android bots. Reverting.

Original issue's description:
> Add support for 'href' (w/o XLink NS) for various SVG elements
> 
> This CL adds a new wrapper type SVGAnimatedHref that wraps 'href' (the
> new, null/default NS version) and '(xlink:)href' (the XLink NS version).
> This wrapper type is used by SVGURIReference and thus most uses of
> 'href' is covered by this part. SVGAnimatedHref is intended as a wrapper
> for accessing the underlying value of either 'href' or 'xlink:href'. Any
> updates due to setAttribute or will go directly to the underlying values
> which means that synchronization etc. does not apply to the wrapper.
> This is one of the "pro"s of this approach - it does not require any
> modification to synching between the SVGString object and the attribute
> in certain cases (like when the "active" attribute is removed). A "con"
> would be that there's a lot of dead space in the object since the
> wrapper itself still needs to implement the SVGAnimatedString interface
> even if just forwards to the actual SVGString.
> The UseCounter for href.baseVal/animVal are moved to the new wrapper.
> 
> For cases where SVGURIReference is not used, new code-paths are added to
> select the right attribute.
> 
> Animation code is updated to target the 'href' in the null/default NS.
> 
> BUG=584142
TBR=alancutter@chromium.org,davve@opera.com,pdr@chromium.org,fs@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584142

[carlosk, 2016-02-12 16:47:15]

Created Revert of Add support for 'href' (w/o XLink NS) for various SVG elements

[commit-bot: I haz the power, 2016-02-12 16:47:37]

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

[carlosk, 2016-02-12 16:47:46]

The CQ bit was unchecked by carlosk@chromium.org

[commit-bot: I haz the power, 2016-02-12 16:48:03]

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

[carlosk, 2016-02-12 16:49:28]

My bad. This CL is probably not the culprit. Removed this revert from the CQ.

[carlosk, 2016-02-12 17:01:56]

Description was changed from

==========
Revert of Add support for 'href' (w/o XLink NS) for various SVG elements
(patchset #6 id:100001 of https://codereview.chromium.org/1681553002/ )

Reason for revert:
This CL seems to be breaking the compilation step of a lot of Android bots.
Reverting.

Original issue's description:
> Add support for 'href' (w/o XLink NS) for various SVG elements
> 
> This CL adds a new wrapper type SVGAnimatedHref that wraps 'href' (the
> new, null/default NS version) and '(xlink:)href' (the XLink NS version).
> This wrapper type is used by SVGURIReference and thus most uses of
> 'href' is covered by this part. SVGAnimatedHref is intended as a wrapper
> for accessing the underlying value of either 'href' or 'xlink:href'. Any
> updates due to setAttribute or will go directly to the underlying values
> which means that synchronization etc. does not apply to the wrapper.
> This is one of the "pro"s of this approach - it does not require any
> modification to synching between the SVGString object and the attribute
> in certain cases (like when the "active" attribute is removed). A "con"
> would be that there's a lot of dead space in the object since the
> wrapper itself still needs to implement the SVGAnimatedString interface
> even if just forwards to the actual SVGString.
> The UseCounter for href.baseVal/animVal are moved to the new wrapper.
> 
> For cases where SVGURIReference is not used, new code-paths are added to
> select the right attribute.
> 
> Animation code is updated to target the 'href' in the null/default NS.
> 
> BUG=584142

TBR=alancutter@chromium.org,davve@opera.com,pdr@chromium.org,fs@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584142
==========

to

==========
Revert of Add support for 'href' (w/o XLink NS) for various SVG elements
(patchset #6 id:100001 of https://codereview.chromium.org/1681553002/ )

Reason for revert:
This CL seems to be breaking the compilation step of a lot of Android bots.
Reverting.

Original issue's description:
> Add support for 'href' (w/o XLink NS) for various SVG elements
> 
> This CL adds a new wrapper type SVGAnimatedHref that wraps 'href' (the
> new, null/default NS version) and '(xlink:)href' (the XLink NS version).
> This wrapper type is used by SVGURIReference and thus most uses of
> 'href' is covered by this part. SVGAnimatedHref is intended as a wrapper
> for accessing the underlying value of either 'href' or 'xlink:href'. Any
> updates due to setAttribute or will go directly to the underlying values
> which means that synchronization etc. does not apply to the wrapper.
> This is one of the "pro"s of this approach - it does not require any
> modification to synching between the SVGString object and the attribute
> in certain cases (like when the "active" attribute is removed). A "con"
> would be that there's a lot of dead space in the object since the
> wrapper itself still needs to implement the SVGAnimatedString interface
> even if just forwards to the actual SVGString.
> The UseCounter for href.baseVal/animVal are moved to the new wrapper.
> 
> For cases where SVGURIReference is not used, new code-paths are added to
> select the right attribute.
> 
> Animation code is updated to target the 'href' in the null/default NS.
> 
> BUG=584142

TBR=alancutter@chromium.org,davve@opera.com,pdr@chromium.org,fs@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584142
==========