Closed Bug 1626575 Opened 4 years ago Closed 4 years ago

Remove Loader.jsm exceptions from Android allowed-dupes.mn

Categories

(DevTools :: Shared Components, task)

task
Not set
normal

Tracking

(firefox76 fixed)

RESOLVED FIXED
Firefox 76
Tracking Status
firefox76 --- fixed

People

(Reporter: ntim, Assigned: aarushivij, Mentored)

Details

(Keywords: good-first-bug)

Attachments

(1 file)

https://hg.mozilla.org/mozilla-central/file/7ef399bfae0d09035e48c8f97dfc9178c3a11f54/mobile/android/installer/allowed-dupes.mn#l8

I've removed some similar ones in https://hg.mozilla.org/mozilla-central/rev/7ef399bfae0d#l2.13 (because those files have been deleted/moved since).

However, Loader.jsm still exists, so I'm wondering if those exceptions are still needed.

It's worth noting that only the mobile allowed-dupes.mn file has this exception.

Julian, do you know if we still need those ?

Flags: needinfo?(jdescottes)

Thanks for the ping! We removed the entries in the other allowed-dupes.mn file in https://bugzilla.mozilla.org/show_bug.cgi?id=1566457 but we missed the Android one.

They should be removed as well.

Flags: needinfo?(jdescottes)

Thanks Julian!

To fix this bug, you'll need to remove:

modules/devtools/Loader.jsm
modules/devtools/shared/Loader.jsm

from mobile/android/installer/allowed-dupes.mn

Mentor: ntim.bugs
Keywords: good-first-bug

Hello, Can I take this bug?
Thanks :)
Aarushi

Flags: needinfo?(ntim.bugs)

Sure, please feel free to submit a patch. Let me know if you've got any questions.

Flags: needinfo?(ntim.bugs)
Summary: Figure out if the Loader.jsm exceptions in allowed-dupes.mn are still needed → Remove Loader.jsm exceptions from Android allowed-dupes.mn
Assignee: nobody → aarushivij
Status: NEW → ASSIGNED
Attachment #9137535 - Attachment description: Bug 1626575 - Figure out if the Loader.jsm exceptions in allowed-dupes.mn are still needed r=ntim → Bug 1626575 - Remove Loader.jsm exceptions from Android allowed-dupes.mn. r=ntim
Pushed by ntim.bugs@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5bea15a54065
Remove Loader.jsm exceptions from Android allowed-dupes.mn. r=ntim
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 76
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: