Add unit test for SVG path parsing

Add unit test for SVG path parsing

This converts svg/dom/path-parser.html into a roughly equivalent unit
test.
Widen the assert in the SVGPathStringSource constructor to allow "" but
not (null) Strings.
Also move the non-fuzzed tests from svg/dom/fuzz-path-parser.html.

BUG=539385
Committed: https://crrev.com/f47f8d15a10dff0acb74bb105aa514e5faf8612c
Cr-Commit-Position: refs/heads/master@{#355008}

[fs, 2015-10-19 15:02:51]

Description was changed from

==========
Add unit test for SVG path parsing

This converts svg/dom/path-parser.html into roughly equivalent unit
tests.
Also move the non-fuzzed tests from svg/dom/fuzz-path-parser.html.

BUG=539385
==========

to

==========
Add unit test for SVG path parsing

This converts svg/dom/path-parser.html into a roughly equivalent unit
test.
Also move the non-fuzzed tests from svg/dom/fuzz-path-parser.html.

BUG=539385
==========

[fs, 2015-10-19 15:02:51]

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

[fs, 2015-10-19 15:57:18]

Description was changed from

==========
Add unit test for SVG path parsing

This converts svg/dom/path-parser.html into a roughly equivalent unit
test.
Also move the non-fuzzed tests from svg/dom/fuzz-path-parser.html.

BUG=539385
==========

to

==========
Add unit test for SVG path parsing

This converts svg/dom/path-parser.html into a roughly equivalent unit
test.
Widen the assert in the SVGPathStringSource constructor to allow "" but
not (null) Strings.
Also move the non-fuzzed tests from svg/dom/fuzz-path-parser.html.

BUG=539385
==========

[pdr., 2015-10-19 18:02:23]

LGTM

https://codereview.chromium.org/1413953002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/svg/SVGPathParserTest.cpp (right):

https://codereview.chromium.org/1413953002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/svg/SVGPathParserTest.cpp:21:
SVGPathStringBuilder builder;               \
Nit: \

[fs, 2015-10-19 18:15:59]

On 2015/10/19 at 18:02:23, pdr wrote:
> LGTM

Damn you're fast! 11 min. before I publish ;-)

>
https://codereview.chromium.org/1413953002/diff/60001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/svg/SVGPathParserTest.cpp (right):
> 
>
https://codereview.chromium.org/1413953002/diff/60001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/svg/SVGPathParserTest.cpp:21:
SVGPathStringBuilder builder;               \
> Nit: \

Meh. Will fix.

[fs, 2015-10-20 08:09:11]

The CQ bit was checked by fs@opera.com

[fs, 2015-10-20 08:09:12]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/1413953002/#ps80001
(title: "Drop line-wrap.")

[commit-bot: I haz the power, 2015-10-20 08:10:02]

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

[commit-bot: I haz the power, 2015-10-20 08:36:34]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-10-20 08:37:28]

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