Clean up SVGViewSpec::parseViewSpecInternal

Clean up SVGViewSpec::parseViewSpecInternal

Make this function slightly more readable by partitioning it as:
  1) Parse outer function ("svgView")
  2) For all functions:
  2.1) Match/parse "function name"
  2.2) Parse arguments to said function

This gets rid of a lot of the parameter list boilerplate that every case
shares. (I.e handling '(' and ')'.)

Also change to use skipExactly and skipUntil from ParsingUtilities.h.

Committed: https://crrev.com/6213989c3e5b0295a6324dd9de3c600512deda04
Cr-Commit-Position: refs/heads/master@{#425315}

[fs, 2016-10-14 12:14:08]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-14 12:14:25]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 13:41:38]

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

[commit-bot: I haz the power, 2016-10-14 13:41:39]

Dry run: This issue passed the CQ dry run.

[fs, 2016-10-14 13:45:39]

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

[Stephen Chennney, 2016-10-14 14:08:06]

Makes it a lot more readable. LGTM.

[fs, 2016-10-14 14:13:01]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-10-14 14:13:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 14:19:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-14 14:21:18]

Description was changed from

==========
Clean up SVGViewSpec::parseViewSpecInternal

Make this function slightly more readable by partitioning it as:
  1) Parse outer function ("svgView")
  2) For all functions:
  2.1) Match/parse "function name"
  2.2) Parse arguments to said function

This gets rid of a lot of the parameter list boilerplate that every case
shares. (I.e handling '(' and ')'.)

Also change to use skipExactly and skipUntil from ParsingUtilities.h.
==========

to

==========
Clean up SVGViewSpec::parseViewSpecInternal

Make this function slightly more readable by partitioning it as:
  1) Parse outer function ("svgView")
  2) For all functions:
  2.1) Match/parse "function name"
  2.2) Parse arguments to said function

This gets rid of a lot of the parameter list boilerplate that every case
shares. (I.e handling '(' and ')'.)

Also change to use skipExactly and skipUntil from ParsingUtilities.h.

Committed: https://crrev.com/6213989c3e5b0295a6324dd9de3c600512deda04
Cr-Commit-Position: refs/heads/master@{#425315}
==========

[commit-bot: I haz the power, 2016-10-14 14:21:19]

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