Enable "browser.tabs.useCache" by default

RESOLVED FIXED in Firefox 46

Status

()

Firefox for Android
General
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Margaret, Assigned: mfinkle)

Tracking

35 Branch
Firefox 46
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox46 fixed, fennec46+)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
This feature landed behind a Nightly-only flag.

This sounds like a good time to use switchboard to release this in a staged rollout and listen for bug reports.
(Reporter)

Updated

2 years ago
tracking-fennec: ? → Nightly+
(Reporter)

Updated

2 years ago
Depends on: 1236130
Created attachment 8709096 [details] [diff] [review]
enable-offline-cache v0.1
Assignee: nobody → mark.finkle
Attachment #8709096 - Flags: review?(margaret.leibovic)
(Reporter)

Comment 2

2 years ago
Comment on attachment 8709096 [details] [diff] [review]
enable-offline-cache v0.1

Review of attachment 8709096 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/app/mobile.js
@@ -32,5 @@
>  pref("browser.tabs.expireTime", 900);
>  
>  // Control whether tab content should try to load from disk cache when network
>  // is offline.
> -#ifdef NIGHTLY_BUILD

Or, if you're a huge advocate for switchboard and staged rollouts, you could put this behind a switchboard control.
Attachment #8709096 - Flags: review?(margaret.leibovic) → review+

Comment 3

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/8415015a072f
(In reply to :Margaret Leibovic from comment #2)

> Or, if you're a huge advocate for switchboard and staged rollouts, you could
> put this behind a switchboard control.

This is a good idea, but we don't have a pattern for using SwitchBoard from JS or for handling prefs yet. I didn't want to rush a pattern in with this bug, but we could followup.
(Reporter)

Comment 5

2 years ago
(In reply to Mark Finkle (:mfinkle) from comment #4)
> (In reply to :Margaret Leibovic from comment #2)
> 
> > Or, if you're a huge advocate for switchboard and staged rollouts, you could
> > put this behind a switchboard control.
> 
> This is a good idea, but we don't have a pattern for using SwitchBoard from
> JS or for handling prefs yet. I didn't want to rush a pattern in with this
> bug, but we could followup.

Someone needs to be the first person to come up with the pattern!

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8415015a072f
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
(Assignee)

Updated

2 years ago
tracking-fennec: Nightly+ → 46+
Depends on: 1258428
You need to log in before you can comment on or make changes to this bug.