Closed Bug 1592134 Opened 3 months ago Closed 3 months ago

Turn on the display default private search engine feature for Nightly users only - correct patch

Categories

(Firefox :: Search, task, P1)

task
Points:
1

Tracking

()

RESOLVED FIXED
Firefox 72
Iteration:
72.1 - Oct 21 - Nov 3
Tracking Status
firefox71 + fixed
firefox72 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

[Tracking Requested - why for this release]:

Bug 1590775 was rebased before landing, and ended up turning off permissions.desktop-notification.postPrompt.enabled rather than browser.search.separatePrivateDefault.ui.enabled.

(Though looking at the history, it looks like it was always disabling the wrong pref, but I'm sure I checked it fully before approving it).

This'll need to be uplifted to Fx 71, since it's enabled on EARLY_BETA at the moment.

cc'ing Johann as this might be affecting their telemetry for the last week.

Assignee: nobody → standard8

Thanks for the heads up!

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7290b4ea908c
Turn on the display of default private search engine feature for Nightly users only. r=johannh

How on earth did this happen?!? Must've been a bad merge(?)

Comment on attachment 9104769 [details]
Bug 1592134 - Turn on the display of default private search engine feature for Nightly users only. r?mikedeboer

Beta/Release Uplift Approval Request

  • User impact if declined: Private browsing wasn't disabled as per bug 1590775. The wrong thing got disabled.

We really don't know how this happened, but we both must have missed the pref being wrong somehow.

  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Pref-only ifdef changes - reverting one to what it was previously, and this time disabling the correct preference for beta (browser.search.separatePrivateDefault.ui.enabled).
  • String changes made/needed: none
Attachment #9104769 - Flags: approval-mozilla-beta?
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 72

Comment on attachment 9104769 [details]
Bug 1592134 - Turn on the display of default private search engine feature for Nightly users only. r?mikedeboer

Low risk, uplift approved for 71 beta 6, thanks.

Attachment #9104769 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.