Closed Bug 1281505 Opened 3 years ago Closed 3 years ago

set network.predictor.prefetch-enabled = true by default only on nightly

Categories

(Core :: Networking, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: u408661, Assigned: u408661)

Details

(Whiteboard: [necko-active])

Attachments

(1 file)

There have been a few issues around prefetch that have caused me to pref it off for the last 2 aurora cycles. Instead of having to do the patch approval & uplift dance every merge day to maintain that state, let's just ifdef the pref to nightly-only until we're more confident in prefetch. Hopefully this will be a short-lived ifdef.
Attached patch patchSplinter Review
Patch to enable by default on nightly, disable by default everywhere else.
Attachment #8764315 - Flags: review?(honzab.moz)
Comment on attachment 8764315 [details] [diff] [review]
patch

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

nit: a bit more readable could be to only ifdef the true/false argument (probably doable only in the cpp file).

r+ for the patch like is or updated.
Attachment #8764315 - Flags: review?(honzab.moz) → review+
Whiteboard: [necko-active]
Pushed by hurley@todesschaf.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ccae0e80b4de
Only enable predictor prefetch by default on nightly. r=mayhemer
https://hg.mozilla.org/mozilla-central/rev/ccae0e80b4de
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.