Fix parsing of exponents in StringToDouble.

Fix parsing of exponents in StringToDouble.

BUG=542881
Committed: https://crrev.com/079cdc8e1483c42d5318a02bd300670cc5b7d477
Cr-Commit-Position: refs/heads/master@{#416158}

[Lei Zhang, 2016-08-30 01:53:10]

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

[commit-bot: I haz the power, 2016-08-30 02:48:59]

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-08-30 03:53:49]

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

[commit-bot: I haz the power, 2016-08-30 03:53:50]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2016-08-30 04:19:45]

thestig@chromium.org changed reviewers:
+ thakis@chromium.org

[Lei Zhang, 2016-08-31 05:36:38]

ping ping

[Lei Zhang, 2016-09-01 23:01:18]

thestig@chromium.org changed reviewers:
+ danakj@chromium.org

[Lei Zhang, 2016-09-01 23:01:19]

chirp chirp chirp

[danakj, 2016-09-01 23:49:24]

LGTM

https://codereview.chromium.org/2296503003/diff/1/base/strings/string_number_...
File base/strings/string_number_conversions_unittest.cc (right):

https://codereview.chromium.org/2296503003/diff/1/base/strings/string_number_...
base/strings/string_number_conversions_unittest.cc:732: {"1.7976e308",
1.7976e308, true},
Can you drop a comment on all these new ones where these numbers come from/what
they are testing?

[Lei Zhang, 2016-09-02 00:56:15]

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

[commit-bot: I haz the power, 2016-09-02 00:56:30]

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

[Lei Zhang, 2016-09-02 01:07:10]

https://codereview.chromium.org/2296503003/diff/1/base/strings/string_number_...
File base/strings/string_number_conversions_unittest.cc (right):

https://codereview.chromium.org/2296503003/diff/1/base/strings/string_number_...
base/strings/string_number_conversions_unittest.cc:732: {"1.7976e308",
1.7976e308, true},
On 2016/09/01 23:49:23, danakj wrote:
> Can you drop a comment on all these new ones where these numbers come
from/what
> they are testing?

- Added comments for all the groups of cases.
- Added a few more test cases.

[danakj, 2016-09-02 01:09:47]

Thanks :) LGTM

[commit-bot: I haz the power, 2016-09-02 01:40:51]

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

[commit-bot: I haz the power, 2016-09-02 01:40:52]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2016-09-02 01:41:09]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2016-09-02 01:41:22]

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-09-02 01:46:31]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-02 01:48:37]

Description was changed from

==========
Fix parsing of exponents in StringToDouble.

BUG=542881
==========

to

==========
Fix parsing of exponents in StringToDouble.

BUG=542881

Committed: https://crrev.com/079cdc8e1483c42d5318a02bd300670cc5b7d477
Cr-Commit-Position: refs/heads/master@{#416158}
==========

[commit-bot: I haz the power, 2016-09-02 01:48:39]

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