1929270 - Use std::clamp instead of min/max chain in dom/

Status: RESOLVED FIXED

(Core :: MFBT, task)

Comment 1

[[:sergesanspaille]]

Attachment: Bug 1929270 - Use std::clamp instead of min/max chain in dom/ r=nika!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a5ae29b17323
Use std::clamp instead of min/max chain in dom/ r=nika

Comment 3

[Norisz Fay [:noriszfay]]

Backed out for causing multiple failures related to mozalloc_abort

Backout link

Push with failures

Failure log // Failure log 2

This failure is also caused by this bug.

Comment 4

[Chris Peterson [:cpeterson]]

The std::clamp assertion failed here:

https://hg.mozilla.org/integration/autoland/rev/a5ae29b17323#l1.16

because StickyTimeDuration(-mEffect->NormalizedTiming().Delay() was less than zeroDuration.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1079b962f439
Use std::clamp instead of min/max chain in dom/ r=nika

Comment 6

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/1079b962f439