It can be sent as a web page creation parameter instead.
Created attachment 361049 [details]
Comment on attachment 361049 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=361049&action=review
Not thrilled with the use of "FD" in the patch.
> + IPC::Attachment hostFD();
This should say "override" or "final"; applies even though the underlying function is a pure virtual. My preference is final, but it seems we have a mix in this file.
> + virtual ~View() = default;
I suggest we consider just deleting this line entirely, because this is what happens if we don’t declare the destructor at all, and I think it’s elegant to just let that happen. OK this way too, though.
> +#if PLATFORM(WPE)
> + virtual IPC::Attachment hostFD() = 0;
This is a messy pattern, but I suppose it’s what we are using for lots of other things above, so it’s not great for me to complain just about this one.
Created attachment 361660 [details]
Patch for landing
Committed r241260: <https://trac.webkit.org/changeset/241260>