Don't report errors when SVG attributes are removed

Don't report errors when SVG attributes are removed

In some cases, SVGAnimated*, and the parser for the underlying object,
will return an error when encountering a null String object (which it
will be passed when the attribute it reflects is removed.) This can be
misleading, so suppress error reporting in these cases.

Also let SVGAnimatedViewBoxRect::setBaseValueAsString delegate to its
superclass rather than open-coding the same thing. This could prove
beneficial if adding additional logic for handling attribute removals.

BUG=523685
Committed: https://crrev.com/a431d862bc9f2a936b31124a127a896f8cfa24f3
Cr-Commit-Position: refs/heads/master@{#375833}

[fs, 2016-02-16 18:12:13]

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

[Stephen Chennney, 2016-02-16 18:22:02]

lgtm

[fs, 2016-02-17 08:50:30]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-02-17 08:51:33]

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

[commit-bot: I haz the power, 2016-02-17 08:56:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-17 08:59:07]

Description was changed from

==========
Don't report errors when SVG attributes are removed

In some cases, SVGAnimated*, and the parser for the underlying object,
will return an error when encountering a null String object (which it
will be passed when the attribute it reflects is removed.) This can be
misleading, so suppress error reporting in these cases.

Also let SVGAnimatedViewBoxRect::setBaseValueAsString delegate to its
superclass rather than open-coding the same thing. This could prove
beneficial if adding additional logic for handling attribute removals.

BUG=523685
==========

to

==========
Don't report errors when SVG attributes are removed

In some cases, SVGAnimated*, and the parser for the underlying object,
will return an error when encountering a null String object (which it
will be passed when the attribute it reflects is removed.) This can be
misleading, so suppress error reporting in these cases.

Also let SVGAnimatedViewBoxRect::setBaseValueAsString delegate to its
superclass rather than open-coding the same thing. This could prove
beneficial if adding additional logic for handling attribute removals.

BUG=523685

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

[commit-bot: I haz the power, 2016-02-17 08:59:08]

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