Fix an assertion failure in InputType::applyStep()

Fix an assertion failure in InputType::applyStep()

If stepBase is extremely larger than step or maximum, the old code produced
alignedMaximum larger than maximum().  This CL fixes it.

- The core logic is moved to StepRange class to improve testability
- Move the stream printer for Decimal because it is used in DCHECK_LE().

BUG=617809
Committed: https://crrev.com/ee4915d796448f4024de2c943a52b007021ead03
Cr-Commit-Position: refs/heads/master@{#399563}

[tkent, 2016-06-08 03:46:25]

Description was changed from

==========
Fix assertion failure in InputType::applyStep()

BUG=617809
==========

to

==========
Fix an assertion failure in InputType::applyStep()

If stepBase is extremely larger than step or maximum, the old code produced
alignedMaximum larger than maximum().  This CL fixes it.

- The core logic is moved to StepRange class to improve testability
- Add stream printer for Decimal because it is used in DCHECK_LE().

BUG=617809
==========

[tkent, 2016-06-08 04:21:29]

Patchset #1 (id:1) has been deleted

[tkent, 2016-06-08 04:21:49]

Description was changed from

==========
Fix an assertion failure in InputType::applyStep()

If stepBase is extremely larger than step or maximum, the old code produced
alignedMaximum larger than maximum().  This CL fixes it.

- The core logic is moved to StepRange class to improve testability
- Add stream printer for Decimal because it is used in DCHECK_LE().

BUG=617809
==========

to

==========
Fix an assertion failure in InputType::applyStep()

If stepBase is extremely larger than step or maximum, the old code produced
alignedMaximum larger than maximum().  This CL fixes it.

- The core logic is moved to StepRange class to improve testability
- Move the stream printer for Decimal because it is used in DCHECK_LE().

BUG=617809
==========

[tkent, 2016-06-08 05:21:25]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2016-06-08 05:21:26]

Keishi, would you review this please? No hurry.

[keishi, 2016-06-13 14:58:05]

LGTM

[tkent, 2016-06-13 16:32:27]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-06-13 16:33:04]

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

[commit-bot: I haz the power, 2016-06-13 17:56:23]

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

[commit-bot: I haz the power, 2016-06-13 17:56:28]

Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[tkent, 2016-06-13 21:22:45]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-06-13 21:23:35]

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

[commit-bot: I haz the power, 2016-06-13 22:00:04]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-06-13 22:00:06]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-13 22:01:41]

Description was changed from

==========
Fix an assertion failure in InputType::applyStep()

If stepBase is extremely larger than step or maximum, the old code produced
alignedMaximum larger than maximum().  This CL fixes it.

- The core logic is moved to StepRange class to improve testability
- Move the stream printer for Decimal because it is used in DCHECK_LE().

BUG=617809
==========

to

==========
Fix an assertion failure in InputType::applyStep()

If stepBase is extremely larger than step or maximum, the old code produced
alignedMaximum larger than maximum().  This CL fixes it.

- The core logic is moved to StepRange class to improve testability
- Move the stream printer for Decimal because it is used in DCHECK_LE().

BUG=617809

Committed: https://crrev.com/ee4915d796448f4024de2c943a52b007021ead03
Cr-Commit-Position: refs/heads/master@{#399563}
==========

[commit-bot: I haz the power, 2016-06-13 22:01:42]

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