Error reporting for number-or-percentage

Error reporting for number-or-percentage

Move parseNumberOrPercentage from SVGParserUtilities to SVGNumber (since
it's the only user) and simplify it.

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

[fs, 2016-02-05 18:52:14]

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

[Stephen Chennney, 2016-02-05 19:00:31]

How does the test result change? I'm assuming messages are better, but I would
like to see it.

[fs, 2016-02-05 19:13:56]

On 2016/02/05 at 19:00:31, schenney wrote:
> How does the test result change? I'm assuming messages are better, but I would
like to see it.

Examples are here:

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

It's not a great test since it's mostly just randomish input. It does show how
the reporting changes though (scroll down because the diff is... not pretty.)

[Stephen Chennney, 2016-02-05 19:43:07]

On 2016/02/05 19:13:56, fs wrote:
> On 2016/02/05 at 19:00:31, schenney wrote:
> > How does the test result change? I'm assuming messages are better, but I
would
> like to see it.
> 
> Examples are here:
> 
>
https://storage.googleapis.com/chromium-layout-test-archives/mac_chromium_rel...
> 
> It's not a great test since it's mostly just randomish input. It does show how
> the reporting changes though (scroll down because the diff is... not pretty.)

Sorry for not thinking of that - I'm rusty.

LGTM

[fs, 2016-02-05 19:56:42]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-02-05 19:57:58]

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

[commit-bot: I haz the power, 2016-02-05 20:11:22]

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

[commit-bot: I haz the power, 2016-02-05 20:11:23]

Internal error: failed to checkout. Please try again.

[commit-bot: I haz the power, 2016-02-05 20:13:30]

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

[fs, 2016-02-05 20:19:57]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-02-05 20:22:55]

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

[commit-bot: I haz the power, 2016-02-05 20:38:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-05 20:39:20]

Description was changed from

==========
Error reporting for number-or-percentage

Move parseNumberOrPercentage from SVGParserUtilities to SVGNumber (since
it's the only user) and simplify it.

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

to

==========
Error reporting for number-or-percentage

Move parseNumberOrPercentage from SVGParserUtilities to SVGNumber (since
it's the only user) and simplify it.

BUG=231612

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

[commit-bot: I haz the power, 2016-02-05 20:39:21]

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