[Webkit-unassigned] [Bug 222366] [WPE][GTK] Reconsider moving network session APIs to WebKitWebsiteDataManager

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jan 20 00:45:21 PST 2023


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

--- Comment #11 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to Michael Catanzaro from comment #9)
> (In reply to Carlos Garcia Campos from comment #7)
> > My initial idea was to move the favicon database ownership from web context
> > to website data manager, but then it can't be shared with ephemeral web
> > views.
> 
> This actually seems ideal. I don't think it's desirable to share the favicon
> database with ephemeral web views because the favicon database will
> necessarily load remote web content, and a server could use the absence of a
> favicon load to determine whether the user has previously visited in
> non-ephemeral mode. It may be a small info leak, but I'd say the server is
> not supposed to be able to do that.
> 

That's a good point, I think we can try to move it then.

> > I think the easiest solution would be to leave the favicon
> > database in the web context and change the way it's configured.
> 
> If you don't want to move it to WebKitWebsiteDataManager, then OK.
> 
> (In reply to Carlos Garcia Campos from comment #8)
> > I'm also considering to remove all (or most of) webkit_web_view_new_with
> > constructors. With all the construct properties we currently have I think
> > most people need to use g_object_new and pass properties. Maybe we can keep
> > with_related_view and add optional parameters to new() for the properties
> > that are expected to be passed (web context, network session and settings,
> > for example)
> 
> Bug #250835

Perfect, let's discuss it there.

-- 
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/20230120/106c9667/attachment-0001.htm>


More information about the webkit-unassigned mailing list