646972 - An interval started in a background tab ends up clamped to the 1Hz clamp forever

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P1)

Description

[Boris Zbarsky [:bzbarsky]]

Steps to reproduce:

1)  Make sure you have new tabs set to open in background.
2)  Open the context menu for the "URL" field link on this bug.
3)  Select "open in new tab"
4)  Wait 10 seconds.
5)  Switch to the new tab.

Expected results: 10 or so numbers around 1000 followed by a whole slew of numbers around 4.

Actual results: all the numbers are about 1000; there is no slew.

The problem is that we clamp the timer before storing mInterval in nsTimeout.  Patch coming up to fix that.  Note that we clam _again_ when using mInterval to reset the timer, so this is exactly what we want: the clamp is only applied when actually setting timers.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Clamp the timeout interval after storing mInterval on the nsTimeout, so interval timers will correctly speed up when a tab goes from background to foreground.

Comment 2

[Boris Zbarsky [:bzbarsky]]

http://hg.mozilla.org/projects/cedar/rev/299615ca371a

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

http://hg.mozilla.org/mozilla-central/rev/299615ca371a