Closed Bug 1558440 Opened 6 years ago Closed 6 years ago

Use SaturatedUint32 for the "visible content during load" heuristic.

Categories

(Core :: DOM: Core & HTML, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

Turns out we do have saturated arithmetic in mfbt!

Type: defect → task

Turns out we do have saturated arithmetic in mfbt, I just missed it.

Also, use just an uint32 for the heuristic. Text length is a uint32 anyway, and
it's unlikely we want to decide anything when the value is over the max uint32
value.

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/eee98afe1bba Use SaturateUint32 for the visible content heuristic. r=bzbarsky
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: