Fix user options for fractional digits in Intl.NumberFormatter

Fix user options for fractional digits in Intl.NumberFormatter

The patch in https://crrev.com/ddb5c2d999c5ee6e31c4a9599bb3ddb293cc3f49
moved all fractional digit settings to default values due to a coding
error. These were not even correct default values, and users observed
errors where percentages were written as "23.0%" instead of "23%".

This patch fixes the setting propagation when appropriate and it changes
the default max fractional digits of a percentage to 0, per spec.

BUG=chromium:544122
R=mnita,jochen
CC=hichris123,adamk
LOG=Y

Committed: https://crrev.com/f5e924eb10060d355e693ce4ce9ce0aaf3a8411f
Cr-Commit-Position: refs/heads/master@{#31468}

[Dan Ehrenberg, 2015-10-22 09:02:00]

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

[commit-bot: I haz the power, 2015-10-22 09:02:16]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1420883002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1420883002/1

[Dan Ehrenberg, 2015-10-22 09:02:24]

Description was changed from

==========
Fix user options for fractional digits in Intl.NumberFormatter

The patch in https://crrev.com/ddb5c2d999c5ee6e31c4a9599bb3ddb293cc3f49
moved all fractional digit settings to default values due to a coding
error. These were not even correct default values, and users observed
errors where percentages were written as "23.0%" instead of "23%".

This patch fixes the setting propagation when appropriate and it changes
the default max fractional digits of a percentage to 0, per spec.

BUG=chromium:544122
R=mnita,hichris123
CC=jochen,adamk
LOG=Y
==========

to

==========
Fix user options for fractional digits in Intl.NumberFormatter

The patch in https://crrev.com/ddb5c2d999c5ee6e31c4a9599bb3ddb293cc3f49
moved all fractional digit settings to default values due to a coding
error. These were not even correct default values, and users observed
errors where percentages were written as "23.0%" instead of "23%".

This patch fixes the setting propagation when appropriate and it changes
the default max fractional digits of a percentage to 0, per spec.

BUG=chromium:544122
R=mnita,jochen
CC=hichris123,adamk
LOG=Y
==========

[Dan Ehrenberg, 2015-10-22 09:02:25]

littledan@chromium.org changed reviewers:
+ jochen@chromium.org
- hichris123@gmail.com

[commit-bot: I haz the power, 2015-10-22 09:23:44]

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

[commit-bot: I haz the power, 2015-10-22 09:23:45]

Dry run: This issue passed the CQ dry run.

[mnita, 2015-10-22 11:00:16]

The CQ bit was checked by mnita@google.com

[mnita, 2015-10-22 11:00:18]

The CQ bit was unchecked by mnita@google.com

[mnita, 2015-10-22 11:00:22]

The CQ bit was unchecked by mnita@google.com

[mnita, 2015-10-22 11:01:18]

The CQ bit was checked by mnita@google.com

[mnita, 2015-10-22 11:01:27]

The CQ bit was unchecked by mnita@google.com

[mnita, 2015-10-22 11:01:56]

On 2015/10/22 09:23:45, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

LGTM
(Looks Good To Me)

[mnita, 2015-10-22 11:02:35]

lgtm

[Dan Ehrenberg, 2015-10-22 11:29:05]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2015-10-22 11:29:13]

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

[commit-bot: I haz the power, 2015-10-22 11:30:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-22 11:31:27]

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