Closed Bug 1746268 Opened 2 years ago Closed 2 years ago

Add site permission strings for android

Categories

(WebExtensions :: Android, defect, P1)

defect

Tracking

(firefox-esr91 unaffected, firefox95 unaffected, firefox96 unaffected, firefox97+ fixed)

RESOLVED FIXED
97 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox95 --- unaffected
firefox96 --- unaffected
firefox97 + fixed

People

(Reporter: mixedpuppy, Assigned: mixedpuppy)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [addons-jira])

Attachments

(2 files)

We need to first address a test failure then fix the strings. We'll do both in this bug.

WARNING -  TEST-UNEXPECTED-FAIL | xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_site_permissions.js | test_site_permissions_have_localization_strings - [test_site_permissions_have_localization_strings : 360] Site permission missing 'midi' - false == true
INFO -  test_site_permissions.js:test_site_permissions_have_localization_strings:360
Keywords: leave-open
Pushed by mlaza@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9782e9bdd383
Skip string verification test on android. CLOSED TREE

Set release status flags based on info from the regressing bug 1741534

:Shane, did you mean to tracking request 96? It shows unaffected and the regressing bug introduced in 97. Also what are the implications of the defect, I ask because it is classified as a P1/S1.

Flags: needinfo?(mixedpuppy)

I don't know why there are any tracking flags. I put it P1/S2 because I need to address this with the right fix today or tomorrow at the latest.

Flags: needinfo?(mixedpuppy)
Pushed by scaraveo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/102c123f2855
add android site permission strings and reenable test r=rpl,flod
Flags: in-testsuite+

Should this still be open?

Flags: needinfo?(mixedpuppy)
Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(mixedpuppy)
Keywords: leave-open
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: