Bug 232396 - Disable NETSCAPE_PLUGIN_API at build time on macOS and Windows
Summary: Disable NETSCAPE_PLUGIN_API at build time on macOS and Windows
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alex Christensen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-27 12:24 PDT by Alex Christensen
Modified: 2021-10-30 10:44 PDT (History)
8 users (show)

See Also:

Attachments
Patch (22.67 KB, patch)
2021-10-27 12:34 PDT, Alex Christensen 		no flags Details | Formatted Diff | Diff
Patch (34.34 KB, patch)
2021-10-27 16:45 PDT, Alex Christensen 		no flags Details | Formatted Diff | Diff
Patch (43.85 KB, patch)
2021-10-27 19:22 PDT, Alex Christensen 		no flags Details | Formatted Diff | Diff
Patch (46.93 KB, patch)
2021-10-27 21:28 PDT, Alex Christensen 		no flags Details | Formatted Diff | Diff
Patch (47.12 KB, patch)
2021-10-27 22:19 PDT, Alex Christensen 		no flags Details | Formatted Diff | Diff
Patch (46.96 KB, patch)
2021-10-28 08:41 PDT, Alex Christensen 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (46.17 KB, patch)
2021-10-28 11:02 PDT, Alex Christensen 		no flags Details | Formatted Diff | Diff
Patch (51.95 KB, patch)
2021-10-28 13:06 PDT, Alex Christensen 		ggaren: review+
Details | Formatted Diff | Diff
Show Obsolete (7) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Christensen 2021-10-27 12:24:08 PDT 
Disable NETSCAPE_PLUGIN_API at build time on macOS
Comment 1 Alex Christensen 2021-10-27 12:34:28 PDT 
Created attachment 442622 [details]
Patch
Comment 2 Darin Adler 2021-10-27 14:58:20 PDT 
Comment on attachment 442622 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=442622&action=review

Looks like there are 10 tests where we have to change expected results or skip or remove the tests.

> Source/WTF/ChangeLog:3
> +        Disable NETSCAPE_PLUGIN_API at build time on macOS

Is it enabled for any other platform, or is this the last one with it on?
Comment 3 Alex Christensen 2021-10-27 16:45:59 PDT 
Created attachment 442651 [details]
Patch
Comment 4 Alex Christensen 2021-10-27 17:34:32 PDT 
Supported was removed on Linux in https://bugs.webkit.org/show_bug.cgi?id=215503

NPAPI worked on Windows WebKitLegacy last I checked but I am under the impression that that was just for Safari on Windows.  I am under the impression that it is no longer used for anything.

It was never made to work for M1 Macs.
It was never made to work for WebKit2 on Windows.
Comment 5 Alex Christensen 2021-10-27 19:22:09 PDT 
Created attachment 442663 [details]
Patch
Comment 6 Alex Christensen 2021-10-27 19:57:34 PDT 
Comment on attachment 442663 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=442663&action=review

> LayoutTests/fast/replaced/pdf-as-embed-with-no-mime-type-is-not-blank.html:-1
> -<!-- webkit-test-runner [ PluginsEnabled=false ] -->

This should load fine.  It looks like some embed loading code is behind NETSCAPE_PLUGIN_API that shouldn't be.
Comment 7 Alex Christensen 2021-10-27 21:28:19 PDT 
Created attachment 442671 [details]
Patch
Comment 8 Alex Christensen 2021-10-27 22:19:30 PDT 
Created attachment 442675 [details]
Patch
Comment 9 Alex Christensen 2021-10-28 08:41:19 PDT 
Created attachment 442708 [details]
Patch
Comment 10 Darin Adler 2021-10-28 09:14:48 PDT 
(In reply to Alex Christensen from comment #4)
> NPAPI worked on Windows WebKitLegacy last I checked

Can we turn that off?
Comment 11 Alex Christensen 2021-10-28 11:02:41 PDT 
Created attachment 442721 [details]
Patch
Comment 12 Alex Christensen 2021-10-28 13:02:59 PDT 
This turns off NPAPI on Windows WebKitLegacy, too.
Comment 13 Alex Christensen 2021-10-28 13:06:36 PDT 
Created attachment 442732 [details]
Patch
Comment 14 Geoffrey Garen 2021-10-28 13:10:37 PDT 
Comment on attachment 442732 [details]
Patch

r=me
Comment 15 Alex Christensen 2021-10-28 14:21:56 PDT 
r285003
Comment 16 Adam Roben (:aroben) 2021-10-30 09:34:42 PDT 
(In reply to Alex Christensen from comment #4)
> It was never made to work for WebKit2 on Windows.

I beg to differ! For example, https://bugs.webkit.org/show_bug.cgi?id=60210

It’s nice to see this all go though.
Comment 17 Alex Christensen 2021-10-30 10:44:09 PDT 
Well there's a blast from the past!  I take that back.  It wasn't implemented for WebKit2 Windows in recent trunk, but it had been implemented many years ago then removed many years ago.