Bug 215503

Summary: [GTK][WPE] Remove support for NPAPI plugins
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, annulen, apinheiro, berto, bugs-noreply, cfleizach, darin, dmazzoni, ews-watchlist, gustavo, gyuyoung.kim, jcraig, jdiggs, mcatanzaro, mkwst, ryuan.choi, samuel_white, sergio
Priority: P2 Keywords: Gtk
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=215565
Attachments:
Description Flags
Patch darin: review+

Carlos Garcia Campos
Reported 2020-08-14 08:03:08 PDT
Now that we have branched for 2.30 it's a good moment to finally remove the plugins support.
Attachments
Patch (202.09 KB, patch)
2020-08-14 08:41 PDT, Carlos Garcia Campos
darin: review+
Carlos Garcia Campos
Comment 1 2020-08-14 08:41:48 PDT
Created attachment 406594 [details] Patch I hope I caught all tests using plugins, I'll watch the bots once the patch lands. I'll also close all bug reports about plugins.
EWS Watchlist
Comment 2 2020-08-14 08:43:18 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Darin Adler
Comment 3 2020-08-14 10:42:01 PDT
Comment on attachment 406594 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=406594&action=review > LayoutTests/platform/glib/TestExpectations:956 > +# Plugins Here’s a follow-up task that’s helpful for clarity and the future, but is a little extra work because of the platforms that still do have the plug-in support around. Seems like it’s worthwhile to *move* these tests all into the plugins and http/tests/plugins directories. Except for the tests that just *happen* to use plug-ins that could be redone to test the same thing without a plug-in. Those we could *copy* into the plugins or http/tests/plugins directory and modify (and possibly rename) the original to not use plug-ins any more. It could be a little bit of a pain to do those moves, but I think it’s good for the health of the project. And it would also make this skipping only two lines long. The alternative is to wait until we can delete plug-in support on all platforms, then we can delete instead of moving.
Carlos Garcia Campos
Comment 4 2020-08-17 05:41:16 PDT
Note You need to log in before you can comment on or make changes to this bug.