Extended error reporting for SVGNumber/Point/Rect and related types

Extended error reporting for SVGNumber/Point/Rect and related types

Add extended error reporting for "number-based" types - SVGNumber,
SVGNumberList, SVGNumberOptionalNumber, SVGPoint, SVGPointList and
SVGRect.

Also clean up some of the parsing functions and eliminate some
unnecessary clear() calls in the list types.

BUG=231612
Committed: https://crrev.com/0d2259ca33d92020651e814dbfe5defe92d3e044
Cr-Commit-Position: refs/heads/master@{#371022}

[fs, 2016-01-22 09:59:49]

Description was changed from

==========
Extended error reporting for SVGNumber/Point/Rect and related types

Add extended error reporting for "number-based" types - SVGNumber,
SVGNumberList, SVGNumberOptionalNumber, SVGPoint, SVGPointList and
SVGRect.

BUG=231612
==========

to

==========
Extended error reporting for SVGNumber/Point/Rect and related types

Add extended error reporting for "number-based" types - SVGNumber,
SVGNumberList, SVGNumberOptionalNumber, SVGPoint, SVGPointList and
SVGRect.

Also clean up some of the parsing functions and eliminate some
unnecessary clear() calls in the list types.

BUG=231612
==========

[fs, 2016-01-22 18:54:55]

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

[fs, 2016-01-22 18:54:56]

Appearances:

https://storage.googleapis.com/chromium-layout-test-archives/mac_chromium_rel...

[pdr., 2016-01-22 19:29:42]

Another nice one.

I wish parse was called parseAndSetValue, but that's more just personal shed
color preferences.

LGTM

[fs, 2016-01-22 19:53:06]

On 2016/01/22 at 19:29:42, pdr wrote:
> Another nice one.
> 
> I wish parse was called parseAndSetValue, but that's more just personal shed
color preferences.

I think my personal shed color would be parse(AndDontSetValue). Same, same, but
different.

[f(malita), 2016-01-22 19:59:30]

LGTM

[pdr., 2016-01-22 19:59:32]

On 2016/01/22 at 19:53:06, fs wrote:
> On 2016/01/22 at 19:29:42, pdr wrote:
> > Another nice one.
> > 
> > I wish parse was called parseAndSetValue, but that's more just personal shed
color preferences.
> 
> I think my personal shed color would be parse(AndDontSetValue). Same, same,
but different.

I like gray, you like grey. Basically the same :) In any case, this patch is
fine as-is.

[fs, 2016-01-22 20:13:23]

On 2016/01/22 at 19:59:32, pdr wrote:
> On 2016/01/22 at 19:53:06, fs wrote:
> > On 2016/01/22 at 19:29:42, pdr wrote:
> > > Another nice one.
> > > 
> > > I wish parse was called parseAndSetValue, but that's more just personal
shed color preferences.
> > 
> > I think my personal shed color would be parse(AndDontSetValue). Same, same,
but different.
> 
> I like gray, you like grey. Basically the same :) In any case, this patch is
fine as-is.

Yes, I am quite the British chap aren't I? Good golly it's late! Time for tea -
maybe Earl Grey. Ta ta!

[fs, 2016-01-22 20:13:54]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-01-22 20:14:26]

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

[commit-bot: I haz the power, 2016-01-22 20:21:10]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-01-22 20:22:13]

Description was changed from

==========
Extended error reporting for SVGNumber/Point/Rect and related types

Add extended error reporting for "number-based" types - SVGNumber,
SVGNumberList, SVGNumberOptionalNumber, SVGPoint, SVGPointList and
SVGRect.

Also clean up some of the parsing functions and eliminate some
unnecessary clear() calls in the list types.

BUG=231612
==========

to

==========
Extended error reporting for SVGNumber/Point/Rect and related types

Add extended error reporting for "number-based" types - SVGNumber,
SVGNumberList, SVGNumberOptionalNumber, SVGPoint, SVGPointList and
SVGRect.

Also clean up some of the parsing functions and eliminate some
unnecessary clear() calls in the list types.

BUG=231612

Committed: https://crrev.com/0d2259ca33d92020651e814dbfe5defe92d3e044
Cr-Commit-Position: refs/heads/master@{#371022}
==========

[commit-bot: I haz the power, 2016-01-22 20:22:14]

Patchset 4 (id:??) landed as
https://crrev.com/0d2259ca33d92020651e814dbfe5defe92d3e044
Cr-Commit-Position: refs/heads/master@{#371022}

[Mitrob77_gmail.com, 2016-01-25 13:33:59]

mitrob77@gmail.com changed reviewers:
+ Mitrob77@gmail.com

[Mitrob77_gmail.com, 2016-01-25 13:34:00]

lgtm