Don't handle out-of-range parameters for arcs in UnalteredParsing mode

Don't handle out-of-range parameters for arcs in UnalteredParsing mode

This means that pathSegList actually keeps/presents the out-of-range
parameters. (This behavior matches Gecko.)

BUG=467592
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191990

[fs, 2015-03-16 18:17:17]

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

[kouhei (in TOK), 2015-03-17 01:57:45]

kouhei@chromium.org changed reviewers:
+ kouhei@chromium.org

[kouhei (in TOK), 2015-03-17 01:57:46]

lgtm

https://codereview.chromium.org/1015433003/diff/1/Source/core/svg/SVGPathPars...
File Source/core/svg/SVGPathParser.cpp (right):

https://codereview.chromium.org/1015433003/diff/1/Source/core/svg/SVGPathPars...
Source/core/svg/SVGPathParser.cpp:253: 
Nit: Might want to add ASSERT(m_pathParsingMode == NormalizedParsing)? Skip this
if you think it is obvious.

[fs, 2015-03-17 09:28:29]

Thanks for the review!

https://codereview.chromium.org/1015433003/diff/1/Source/core/svg/SVGPathPars...
File Source/core/svg/SVGPathParser.cpp (right):

https://codereview.chromium.org/1015433003/diff/1/Source/core/svg/SVGPathPars...
Source/core/svg/SVGPathParser.cpp:253: 
On 2015/03/17 01:57:46, kouhei wrote:
> Nit: Might want to add ASSERT(m_pathParsingMode == NormalizedParsing)? Skip
this
> if you think it is obvious.

With the follow-ups I have this is going to be more than obvious (code-paths
will diverge at a higher level), so I'll punt on this assert.

[fs, 2015-03-17 09:28:33]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2015-03-17 09:29:00]

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

[commit-bot: I haz the power, 2015-03-17 09:32:03]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191990