Closed
Bug 614733
Opened 14 years ago
Closed 14 years ago
Do exponential backoff on background tab refresh drivers
Categories
(Core :: Layout, defect, P2)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla5
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
Details
Attachments
(2 files, 2 obsolete files)
|
5.61 KB,
patch
|
dbaron
:
review+
|
Details | Diff | Splinter Review |
|
855 bytes,
patch
|
Details | Diff | Splinter Review |
roc suggested this earlier today. Seems like an excellent idea to me. David, any objections?
|
Assignee | |
Comment 1•14 years ago
|
||
|
|
Assignee | |
Updated•14 years ago
|
Attachment #493175 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 2•14 years ago
|
||
|
|
Assignee | |
Updated•14 years ago
|
Attachment #493176 -
Flags: review?(dbaron)
|
|
Assignee | |
Updated•14 years ago
|
Assignee: nobody → bzbarsky
What makes us start a repeating timer again when we go from throttled to unthrottled? Maybe SetThrottled should stop and restart the timer (always) just like Notify does (when mThrottled)?
Other than that this looks fine, though.
|
|
Assignee | |
Comment 4•14 years ago
|
||
> What makes us start a repeating timer again when we go from throttled to
> unthrottled?
Nothing. Good catch.
I'll fix it as you propose.
|
|
Assignee | |
Comment 5•14 years ago
|
||
Actually, I'll just do that when going from throttled to not throttled.
|
|
Assignee | |
Updated•14 years ago
|
Attachment #493176 -
Attachment is obsolete: true
Attachment #493176 -
Flags: review?(dbaron)
|
|
Assignee | |
Comment 6•14 years ago
|
||
Actually, I just made it always stop/restart
Attachment #499860 -
Flags: review?(dbaron)
|
|
Assignee | |
Comment 7•14 years ago
|
||
Comment on attachment 499860 [details] [diff] [review]
Do exponential backoff on throttled refresh drivers.
r=dbaron
Attachment #499860 -
Flags: review?(dbaron) → review+
|
|
Assignee | |
Updated•14 years ago
|
Whiteboard: [need review] → [need gk2 ship]
|
|
Assignee | |
Comment 9•14 years ago
|
||
Whiteboard: [need gk2 ship] → fixed-in-birch
Status: NEW → RESOLVED
Closed: 14 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Whiteboard: fixed-in-birch
Target Milestone: --- → mozilla2.2
|
|
Assignee | |
Updated•14 years ago
|
Flags: in-testsuite?
Depends on: 645943
You need to log in
before you can comment on or make changes to this bug.
Description
•