Remove InterpolableBool and existing use of it in SVG path interpolation

Remove InterpolableBool and existing use of it in SVG path interpolation

The InterpolableBool class should not be used as bools are not considered
interpolable. This change replaces the single use of it with
InterpolableNumber. This retains the existing behaviour but is not
ideal.

This change fixes a nullptr crash caused by the use of InterpolableBool.

BUG=662694
Committed: https://crrev.com/2439d975210aba96119d8b75633b9f5659563d00
Cr-Commit-Position: refs/heads/master@{#430907}

[alancutter (OOO until 2018), 2016-11-09 06:41:28]

alancutter@chromium.org changed reviewers:
+ ericwilligers@chromium.org

[Eric Willigers, 2016-11-09 08:10:25]

The CQ bit was checked by ericwilligers@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 08:10:42]

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-11-09 10:27:17]

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

[commit-bot: I haz the power, 2016-11-09 10:27:18]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2016-11-09 10:42:50]

The CQ bit was checked by ericwilligers@chromium.org

[commit-bot: I haz the power, 2016-11-09 10:43:01]

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-11-09 10:43:04]

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

[commit-bot: I haz the power, 2016-11-09 10:43:05]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Eric Willigers, 2016-11-09 11:02:43]

The CQ bit was checked by ericwilligers@chromium.org

[Eric Willigers, 2016-11-09 11:02:43]

lgtm

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

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-11-09 11:06:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-09 11:08:58]

Description was changed from

==========
Remove InterpolableBool and existing use of it in SVG path interpolation

The InterpolableBool class should not be used as bools are not considered
interpolable. This change replaces the single use of it with
InterpolableNumber. This retains the existing behaviour but is not
ideal.

This change fixes a nullptr crash caused by the use of InterpolableBool.

BUG=662694
==========

to

==========
Remove InterpolableBool and existing use of it in SVG path interpolation

The InterpolableBool class should not be used as bools are not considered
interpolable. This change replaces the single use of it with
InterpolableNumber. This retains the existing behaviour but is not
ideal.

This change fixes a nullptr crash caused by the use of InterpolableBool.

BUG=662694

Committed: https://crrev.com/2439d975210aba96119d8b75633b9f5659563d00
Cr-Commit-Position: refs/heads/master@{#430907}
==========

[commit-bot: I haz the power, 2016-11-09 11:08:59]

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