[Webkit-unassigned] [Bug 243659] [GTK][WPE] Expose notifications without a page in public API

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 8 05:32:52 PDT 2022


https://bugs.webkit.org/show_bug.cgi?id=243659

Michael Catanzaro <mcatanzaro at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mcatanzaro at gnome.org

--- Comment #1 from Michael Catanzaro <mcatanzaro at gnome.org> ---
(In reply to Carlos Garcia Campos from comment #0) 
> I think it's probably better to add the global singleton to handle all
> notifications with the web view passed as a parameter.

I agree. Of course, this would not break the existing WebKitWebView API that you would deprecated.

The downside is this would be our first global API that is not associated with a WebKitWebContext, which breaks the illusion that it's a true library context object and you can have separate instances of WebKit running if you have multiple WebKitWebContext objects. That doesn't seem to be avoidable, but maybe we could design it so that it looks like it's a property of the WebKitWebContext even though it's not really? We could have webkit_web_context_get_notification_manager() and it would just return the same WebKitNotificationManager regardless of which WebKitWebContext you are using, but maybe 10 years from now if internals change it would be possible to revisit that without needing another API break. On the other hand, maybe that's overengineering.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220808/fe34e28d/attachment.htm>


More information about the webkit-unassigned mailing list