Closed
Bug 1185324
Opened 11 years ago
Closed 5 years ago
Duplicate MIME type entries can be shown if multiple libraries are pointing to a plugin
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: sworddragon2, Unassigned)
Details
(Whiteboard: [lang=js] [good next bug])
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0
Build ID: 20150629110007
Steps to reproduce:
1. Install 2 libraries for a plugin (in my case a 64 bit Flash Player and the 32 bit debug version of it (with nspluginwrapper)).
2. Go to the menu bar -> Tools -> Add-ons -> Plugins and click "More" of the specific plugin.
Actual results:
For example in my case the MIME types are shown 2 times.
Expected results:
Duplicate MIME types should not be shown.
Hi sworddragon2,
Try updating to Firefox 42, are you still having the issue on the newest build? Thanks.
Flags: needinfo?(sworddragon2)
Comment 2•10 years ago
|
||
This is still an issue.
The affected code is here:
https://dxr.mozilla.org/mozilla-central/rev/2bdd9ec79799eff3ceec0a318f5a0632d918a527/toolkit/mozapps/extensions/internal/PluginProvider.jsm#459
The problem is that we collect all of the mimetypes into the array |types|, without filtering out duplicates.
We should use a Set [0] here and |return [...types]| from this function.
We need to do some manual testing of this to check it is working.
0: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set
Mentor: gfritzsche
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(sworddragon2)
Whiteboard: [lang=js] [good next bug]
Updated•10 years ago
|
OS: Linux → All
Hardware: x86_64 → All
Comment 3•5 years ago
|
||
gfritzsche has not been involved with Mozilla for more than a year so will unfortunately no longer be a suitable mentor.
Mentor: gfritzsche
Comment 4•5 years ago
|
||
Resolving as wont fix, plugin support deprecated in Firefox 85.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Updated•4 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•