perf: remove assignDeterministic verbose Math.ceil code to improve performance

[alexzhang1030]

…ance

What kind of change does this PR introduce?

performance

Did you add tests for your changes?

No need

Does this PR introduce a breaking change?

No.

What needs to be documented once your changes are merged?

No.

Introduce about this PR

The old code about assignDeterministicIds fill rate is:

Math.ceil(items.length * 1.25)

but in this PR, it changed to

Math.ceil(items.length * 20)

Because items is an array, and its length will never be a decimal. So it's not necessary to add Math.ceil now.

I removed Math.ceil to improve the performance.

And this change will not cause any Breaking Change, I ran all tests.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: alexzhang1030 (61dee6e)

[webpack-bot]

For maintainers only:

• This needs to be documented (issue in webpack/webpack.js.org will be filed when merged)
• This needs to be backported to webpack 4 (issue will be created when merged)

[alexzhang1030]

@alexander-akait hey, may I ask you how to fix this test? In my local, it's passed.

[alexander-akait]

Please ignore the fail (random fail)

[sokra]

Thanks