Rename local limit variable in SVG transform argument parsing

Rename local limit variable in SVG transform argument parsing

Rename maxPossibleParams to requiredWithOptional to more precisely
describe what it is.

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

[fs, 2016-02-02 11:19:49]

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

[pdr., 2016-02-02 21:17:44]

On 2016/02/02 at 11:19:50, fs wrote:
> 

LGTM with or without the suggestion below.

The last conditional might be easier to understand with a DeMorgan upgrade.
Something like so:
if (arguments.size() != required && arguments.size() != requiredWithOptional)
    return NaughtyArgumentCount;

[fs, 2016-02-02 22:06:56]

On 2016/02/02 at 21:17:44, pdr wrote:
> On 2016/02/02 at 11:19:50, fs wrote:
> > 
> 
> LGTM with or without the suggestion below.
> 
> The last conditional might be easier to understand with a DeMorgan upgrade.
Something like so:
> if (arguments.size() != required && arguments.size() != requiredWithOptional)
>     return NaughtyArgumentCount;

Since I'll probably wait for https://codereview.chromium.org/1659513002 to land
first I might as well...

[fs, 2016-02-03 10:31:56]

The CQ bit was checked by fs@opera.com

[fs, 2016-02-03 10:31:56]

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/1655153002/#ps20001
(title: "DeMorganize")

[commit-bot: I haz the power, 2016-02-03 10:33:04]

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

[commit-bot: I haz the power, 2016-02-03 11:37:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-03 11:37:58]

Description was changed from

==========
Rename local limit variable in SVG transform argument parsing

Rename maxPossibleParams to requiredWithOptional to more precisely
describe what it is.

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

to

==========
Rename local limit variable in SVG transform argument parsing

Rename maxPossibleParams to requiredWithOptional to more precisely
describe what it is.

BUG=231612

Committed: https://crrev.com/4b7b7876805e6a8339ee82f743306c310a88fc1b
Cr-Commit-Position: refs/heads/master@{#373217}
==========

[commit-bot: I haz the power, 2016-02-03 11:37:59]

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