Closed Bug 548680 Opened 15 years ago Closed 3 years ago

Plugins: Make PBrowserStream/PPluginStream stateful, with non-racing destructors

Categories

(Core Graveyard :: Plug-ins, enhancement, P5)

enhancement

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: cjones, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

Attachments

(1 file)

Attached patch Proposed PBrowserStream (deleted) — Splinter Review
I'm using "review?" where I really mean "feedback?". I don't intend to commit this. The attached protocol captures as well as I can my understanding of browser streams. I prefer this more explicitly stateful protocol because I think NPAPI developers (may their numbers ever dwindle) could find it to be useful documentation. All the messages are async for no reason in particular, mostly as an IPDL type-checker testcase. (Again, this is just a prototype.) This protocol will pass the IPDL type checker after |discard| (for which I have a frontend impl) and another semantics bug land.
Attachment #429020 - Flags: review?(benjamin)
Attachment #429020 - Attachment is patch: true
Attachment #429020 - Attachment mime type: application/octet-stream → text/plain
Comment on attachment 429020 [details] [diff] [review] Proposed PBrowserStream Cancelling review, I have an impl that basically does this.
Attachment #429020 - Flags: review?(benjamin)
Component: IPC → Plug-ins
Priority: -- → P5
Resolving as wont fix, plugin support deprecated in Firefox 85.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: