[builtins] Drop the special MathRandomRaw.

[builtins] Drop the special MathRandomRaw.

This is the last user of %_DoubleLo and actually unnecessary, since we
can just use the normal Math.random() instead.

R=yangguo@chromium.org

Committed: https://crrev.com/50e19547038bf2510bd750853560e7a20623cdb0
Cr-Commit-Position: refs/heads/master@{#37359}

[Yang, 2016-06-29 05:53:42]

LGTM with comment.

https://codereview.chromium.org/2109483004/diff/1/src/js/collection.js
File src/js/collection.js (right):

https://codereview.chromium.org/2109483004/diff/1/src/js/collection.js#newcod...
src/js/collection.js:116: hash = (MathRandom() * 0x3FFFFFFF) | 0;
MathRandom returns everything from 0 inclusive to 1 exclusive. So this should
actually multiply with 0x40000000.

[Benedikt Meurer, 2016-06-29 05:54:41]

https://codereview.chromium.org/2109483004/diff/1/src/js/collection.js
File src/js/collection.js (right):

https://codereview.chromium.org/2109483004/diff/1/src/js/collection.js#newcod...
src/js/collection.js:116: hash = (MathRandom() * 0x3FFFFFFF) | 0;
On 2016/06/29 05:53:42, Yang wrote:
> MathRandom returns everything from 0 inclusive to 1 exclusive. So this should
> actually multiply with 0x40000000.

Done.

[Benedikt Meurer, 2016-06-29 05:54:44]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2016-06-29 05:54:45]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2109483004/#ps20001
(title: "Fix off-by-one.")

[commit-bot: I haz the power, 2016-06-29 05:54:52]

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-06-29 06:19:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-29 06:21:09]

Description was changed from

==========
[builtins] Drop the special MathRandomRaw.

This is the last user of %_DoubleLo and actually unnecessary, since we
can just use the normal Math.random() instead.

R=yangguo@chromium.org
==========

to

==========
[builtins] Drop the special MathRandomRaw.

This is the last user of %_DoubleLo and actually unnecessary, since we
can just use the normal Math.random() instead.

R=yangguo@chromium.org

Committed: https://crrev.com/50e19547038bf2510bd750853560e7a20623cdb0
Cr-Commit-Position: refs/heads/master@{#37359}
==========

[commit-bot: I haz the power, 2016-06-29 06:21:10]

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