Closed Bug 1529133 Opened 5 years ago Closed 3 years ago

[meta] Remove Fake Plugins

Categories

(Core Graveyard :: Plug-ins, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: qdot, Unassigned)

References

Details

(Keywords: meta)

Fake plugins were originally used for JS-implemented plugins, like PDFjs and Shumway. AFAICT, these days they're mostly dead code, and we're moving toward total deprecation of plugins anyways. A couple of telemetry tests uses fake plugin tags, but that's about it.

Removing fake plugins as a whole make take a bit, just because there's a decent amount of code involved. However, there's a lot of logic branches around the code pertaining to FakePlugin rendering that could be cleaned up along the way to this goal. This bug will track removing pieces of them as we can.

:jimm ni'ing to make sure I'm not off the mark here. We're not using FakePlugins anywhere that I can see, but not sure if I'm missing something.

Flags: needinfo?(jmathies)

Posted an Intent To Unship on dev-platform, and talked to peterv about potential usages. Only use case that comes up is pdfjs, and no one seems to feel like fake plugins were the solution for that, so I'm going to go ahead with this. We can always back out the removal as needed, since a lot of this code isn't really in oft-changed areas anyways.

Flags: needinfo?(jmathies)
Assignee: kyle → nobody
Depends on: 1678110
Resolving as wont fix, plugin support deprecated in Firefox 85.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
See Also: → 1859416
You need to log in before you can comment on or make changes to this bug.