1762733 - Remove obsolete plugin code from SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: General, task)

Description

[Frank-Rainer Grahl (:frg)]

Clean out remaining NPAPI plugin code from suite.

We need to port bits and pieces from various bugs for this. Most notable:

Bug 1675349 Disable plugin support
Bug 1682030 Mass removal of NPAPI plugin code
Bug 1706452 Package libmozgtk.so for Linux builds

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-plugincleanup-25312.patch

With all the bugs it touches this was like peeling an apple so sorry for the one big patch. But 90% is trivial.
The only code worth looking at is in suite/components/bindings/notification.xml. But even this is basically removal only and I tested extensively. There is still some plugin-crashed code in which will never be executed. In the unlikely event that we ever support EME we need to fix this up. I think lets leave it in for now so that we know what to touch.
suite/components/pref/content/pref-applications.js is the other file worth a look but also tested. In the unlikely event that someone has a handler set for a plugin there is fallback code in with kActionUsePlugin defined locally. Basically the same as in Firefox.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: Much dead code
Testing completed (on m-c, etc.): 2.53.12b1 pre
Risk to taking this patch (and alternatives if risky):
String changes made by this patch: various removals and 2 new variables accounting for the plugin prefs removal.

Comment 2

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-plugincleanup-tests-25312.patch

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: broken tests
Testing completed (on m-c, etc.): already broken
Risk to taking this patch (and alternatives if risky): test only.
String changes made by this patch: --

Comment 3

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-removeflashblocking-25312.patch

This is for our mozilla release branch only. Removal of the flash cassifier lists which are only good as a virtual doorstopper at this point in time. Tested with both SeaMonkey and a Firefox built from our 2.53 branch.

[Approval Request Comment]
Regression caused by (bug #): --
User impact if declined: Useless list fetches.
Testing completed (on m-c, etc.): 2.53.12b1 pre / 56.12
Risk to taking this patch (and alternatives if risky): cleanup code.
String changes made by this patch: --

Comment 4

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-removepluginprefs-25312.patch

Again stuff for our mozilla release branch only.
Bits and pieces removed from various bugs we will never port because they touch removed code but some of this is still in central like the flashclassification. As far as I see it this is dead as are most of the remaining dom.ipc.plugins.* preferences:
https://searchfox.org/mozilla-central/search?q=flashclassi&path=&case=false&regexp=false

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:

Comment 5

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-suite-package-25312.patch

Trivial packaging fix porting needed part of bug 1706452 "Package libmozgtk.so for Linux builds" r/a=me

Comment 6

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-plugincleanup-mail-25312.patch

Stumbled over some mail code. Will never be used by us but r/a=me for our 2.53 and 2.57 branches only to make the cleanup complete.

Comment 7

[Ian Neal]

Comment on attachment 9270565 [details] [diff] [review]
1762733-plugincleanup-25312.patch

[Triage Comment]
Shouldn't plugin.png also be removed from both themes?
r/a=me with that addressed

Comment 8

[Ian Neal]

Comment on attachment 9270566 [details] [diff] [review]
1762733-plugincleanup-tests-25312.patch

[Triage Comment]
LGTM r/a=me

Comment 9

[Ian Neal]

Comment on attachment 9270567 [details] [diff] [review]
1762733-removeflashblocking-25312.patch

[Triage Comment]
LGTM r/a=me

Comment 10

[Ian Neal]

Comment on attachment 9270568 [details] [diff] [review]
1762733-removepluginprefs-25312.patch

[Triage Comment]
LGTM r/a=me

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/eb6738538ba0
Wipe defunct plugin tests. r=IanN
https://hg.mozilla.org/comm-central/rev/3d443503387f
Remove obsolete plugin code. r=IanN
https://hg.mozilla.org/comm-central/rev/f50ef94b27fc
Port bug 1706452: Package libmozgtk.so for Linux builds. r=me

Comment 12

[Frank-Rainer Grahl (:frg)]

[Triage Comment]
Shouldn't plugin.png also be removed from both themes?
r/a=me with that addressed

Yes not used. Removed in checkin.

Comment 13

[Frank-Rainer Grahl (:frg)]

Attachment: 1762733-plugincleanup-mail-25312.patch

Looks like some of the mail sutff was still in central and ended up just being removed now. So just some stuff left for our branch removed earlier which I didn't track where.

Not really used in our code so as before r/a=me

Comment 14

[Frank-Rainer Grahl (:frg)]

Stuff checked in for 2.53.12:

https://gitlab.com/seamonkey-project/seamonkey-2.53-mozilla/-/commit/143e0108583083eb85117a5854a5adf3517494d9
Remove obsolete plugin prefs and code. r=IanN a=IanN
https://gitlab.com/seamonkey-project/seamonkey-2.53-mozilla/-/commit/7d621dd0d5fb4d3c1f72cd2d578203bd4705fee7
Remove obsolete flash blocklists. r=IanN a=IanN

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/8b0b1bf890b4baa34a1bf029601c011e7f9e68ff
Remove some obsolete plugin test code. r=me a=me
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/0d82513ecff9c23bb1987170590f6e29990d06c7
Wipe defunct plugin tests. r=IanN a=IanN
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/20152350948af6dbb8765c8b571e818b3540d91e
Remove obsolete plugin code. r=IanN a=IanN
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/472a4979c5371edacc8ec22f4b3ee8dc54d4a994
Port bug 1706452: Package libmozgtk.so for Linux builds. r=me a=me

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/577ac7fbce27
Removed unused plugin icons from modern theme. r=me

Comment 16

[Frank-Rainer Grahl (:frg)]

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/d762759206f54bff1cd0ecbbdccbd11921b5ac07
Removed unused plugin icons from modern theme. r=me a=me