Closed Bug 1721700 Opened 3 years ago Closed 3 years ago

Make the pref browser.tabs.unloadOnLowMemory dynamic

Categories

(Firefox :: Tabbed Browser, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: toshi, Assigned: toshi)

References

Details

Attachments

(1 file)

Currently changing the pref browser.tabs.unloadOnLowMemory requires process restart. This is a task to make the pref dynamic (= changeable in the middle of session) to make the Normandy experiment easier.

We have the pref browser.tabs.unloadOnLowMemory to turn on the tab unloading
feature. Since it controlled whether we initialize the memory watcher or not
in TabUnloader.init(), changing it required process start.

To conduct A/B testing for this feature, our infrastructure needs to be able
to change the pref in the middle of session. So this change moves the pref
check from TabUnloader.init() to TabUnloader.unloadTabAsync(). With this,
we fully exercise the memoruy watcher regardless of the pref's value, but invokes
tab unloading or memory-pressure only when the pref is on.

Pushed by tkikuchi@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a8d9e102189a Move browser.tabs.unloadOnLowMemory to TabUnloader.unloadTabAsync. r=haik
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: