Closed Bug 1623173 Opened 4 years ago Closed 4 years ago

Preserve embedderDisabled when rebuilding addon DB

Categories

(GeckoView :: Extensions, defect, P1)

Unspecified
All
defect

Tracking

(firefox75 verified, firefox76 verified)

VERIFIED FIXED
mozilla76
Tracking Status
firefox75 --- verified
firefox76 --- verified

People

(Reporter: agi, Assigned: agi)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [geckoview:m76])

Attachments

(1 file)

:rpl helped me with this. Right now when upgrading major versions we lose that property and the addons are enabled, see: https://github.com/mozilla-mobile/android-components/issues/6255

To verify the fix:

install Fennec
Install a non-supported addon like Privacy Badger
Update to a Fenix version that's on Gecko 75
start fenix, verify privacy badger is not enabled
Update to latest fenix (so Gecko 76)
Notice privacy badger is disabled (right now it's enabled)

Pushed by asferro@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/86830c2de974
Preserve embedderDisabled when rebuilding addon DB. r=mixedpuppy
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76

Comment on attachment 9133978 [details]
Bug 1623173 - Preserve embedderDisabled when rebuilding addon DB.

Beta/Release Uplift Approval Request

  • User impact if declined: Some addons will be enabled in Fenix that should not be enabled.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • 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): Just adding a new field to the db migration code
  • String changes made/needed:
Attachment #9133978 - Flags: approval-mozilla-beta?

Comment on attachment 9133978 [details]
Bug 1623173 - Preserve embedderDisabled when rebuilding addon DB.

fix an issue with addons on fenix, approved for 75.0b6

Attachment #9133978 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Hi! I could reproduce this on Pixel 2 (Android 9) and Huawei P9 Lite (Android 7) following the steps:

  1. Install Fennec Nightly;
  2. Install unsupported add-ons, Privacy Badger, Ghostery, NoScript;
  3. Migrate to Fenix Nightly from 3/4 GV 75;
  4. Update to a build from 3/15 GV 76;

Using the latest Fennec Nightly Migration build from 3/23 GV 76, the unsupported add-ons are not active and are not displayed in the menu.
Due to this I will mark this as verified on Nightly GV 76.

This is not reproducible on Beta migration 75.0.0-beta.2 GV 75 from 3/25 with Pixel 2 (Android 9) and Huawei P9 Lite (Android 7).
Due to this I will mark this as Verified.

Status: RESOLVED → VERIFIED
Has Regression Range: --- → yes
Keywords: regression
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: