Ordinary background throttling should be more aggressive

RESOLVED FIXED in Firefox 56

Status

()

Core
DOM
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: farre, Assigned: farre)

Tracking

unspecified
mozilla56
Points:
---

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

11 months ago
Bug 1362322 added means to avoid throttling background tabs on a bunch of conditions. This turns out to be too aggressive to use for all background throttling. Bug 1378124 does it the right way in that it only does it for when we actually want to consider budget throttling. Also, the fallback should be to behave as ordinary throttling when budget throttling isn't allowed.
(Assignee)

Comment 2

11 months ago
Created attachment 8888347 [details] [diff] [review]
0001-Bug-1382254-Only-disallow-budget-throttling.-r-ehsan.patch

Ehsan, this does what we talked about yesterday. Re-instate the more aggressive throttling for ordinary background throttling (the one every second throttling).

It disallows budget throttling for:
IndexedDB, WebSockets, GetUserMedia, Audio.
Assignee: nobody → afarre
Flags: needinfo?(ehsan)
Attachment #8888347 - Flags: review?(ehsan)

Updated

11 months ago
Flags: needinfo?(ehsan)

Updated

11 months ago
Attachment #8888347 - Flags: review?(ehsan) → review+

Comment 3

11 months ago
Why IndexedDB?  Should we exempt other storage users?

Comment 5

11 months ago
Pushed by afarre@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7ca61520bf6d
Only disallow budget throttling. r=ehsan
(Assignee)

Comment 6

11 months ago
(In reply to Ben Kelly [:bkelly] from comment #3)
> Why IndexedDB?  Should we exempt other storage users?

It's for when there are ongoing transactions only, and it's to have the same behaviour as Chrome.

Comment 7

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/7ca61520bf6d
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.