Closed Bug 1531540 Opened 8 months ago Closed 8 months ago

Don't specialise Nursery::SubChunkLimit for mobile.

Categories

(Core :: JavaScript: GC, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: pbone, Assigned: pbone)

References

Details

Attachments

(1 file)

This was specialised so that it wouldn't crash on mobile wouldn't crash when:
SubChunkLimit + SubChunkStepstep > NurseryChunkUsableSize becase
NurseryChunkUsableSize is smaller on mobile.

However Bug 1530575 reduced the value of SubChunkStep so now we no-longer
need to specialise SubChunkLimit. We never tested the impact of
SubChunkLimit on mobile so it's just simpler to have one value.

This was specialised so that it wouldn't crash on mobile wouldn't crash
when: SubChunkLimit + SubChunkStepstep > NurseryChunkUsableSize becase
NurseryChunkUsableSize is smaller on mobile.

However Bug 1530575 reduced the value of SubChunkStep so now we no-longer
need to specialise SubChunkLimit. We never tested the impact of
SubChunkLimit on mobile so it's just simpler to have one value.

Assignee: nobody → pbone
Status: NEW → ASSIGNED
Priority: -- → P3
Pushed by pbone@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ab0fd75fd142
Don't specialise SubChunkLimit for mobile r=sfink
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.