Bug 232462

Summary: Remove code inside ENABLE(NETSCAPE_PLUGIN_API)
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: New BugsAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, annulen, benjamin, bfulgham, cdumez, changseok, cmarcelo, eric.carlson, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, jer.noble, philipj, ryuan.choi, sergio, ysuzuki
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=232519
https://bugs.webkit.org/show_bug.cgi?id=270068
Bug Depends on:    
Bug Blocks: 232642, 237555    
Attachments:
Description Flags
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch ews-feeder: commit-queue-

Description Alex Christensen 2021-10-28 16:28:22 PDT
Remove code inside ENABLE(NETSCAPE_PLUGIN_API)
Comment 1 Alex Christensen 2021-10-28 16:30:00 PDT
Created attachment 442764 [details]
Patch
Comment 2 Alex Christensen 2021-10-28 17:59:35 PDT
Created attachment 442769 [details]
Patch
Comment 3 Anders Carlsson 2021-10-28 19:27:35 PDT
Comment on attachment 442769 [details]
Patch

I have to say I'm a little sad I didn't get to pull the plug(-ins), but at least I can r=me doing so.
Comment 4 Alex Christensen 2021-10-28 20:44:15 PDT
Created attachment 442777 [details]
Patch
Comment 5 Alex Christensen 2021-10-28 22:44:56 PDT
iOS needs a clean build after this change.
Comment 6 Yusuke Suzuki 2021-10-29 00:39:00 PDT
I'm pretty happy that we can remove NPAPI support!!!
Now, super hacky RuntimeArray thing (runtime_array.cpp) is only used by Objective-C bridge.
Comment 7 Alex Christensen 2021-10-29 13:33:34 PDT
r285047
Comment 8 Brent Fulgham 2021-10-29 14:09:33 PDT
I love this patch so much!
Comment 9 Alex Christensen 2021-10-29 14:11:11 PDT
r285051
Comment 10 Alex Christensen 2021-10-29 14:15:54 PDT
r285052
Comment 11 Alex Christensen 2021-10-29 15:21:32 PDT
r285058
Comment 12 Alex Christensen 2021-10-29 16:06:38 PDT
Internal commit 2e4c44c95333771d6b86b5ac3f93cc5edb845ac2 was also necessary to fix the internal build.
Comment 13 Brent Fulgham 2021-11-12 10:18:43 PST
<rdar://problem/84566739>