[Webkit-unassigned] [Bug 125582] Implement RemoteNetworkingContext::initiatingPageID() for Soup

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 9 13:35:32 PST 2014


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





--- Comment #4 from Anders Carlsson <andersca at apple.com>  2014-01-09 13:33:15 PST ---
(In reply to comment #3)
> (In reply to comment #2)
> > (From update of attachment 218973 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=218973&action=review
> 
> Thanks for looking at the patch.
> 
> > I don't understand why a page ID is needed by custom protocol handlers. Isn't is a layering violation for protocol handlers to have any idea of "web page"?
> 
> It allows to identify the web page that initiated the load, since custom protocols are handled globally by the context, it's not possible to know which web page is associated to the custom protocol load. This is the same concept that the initiating page of a download operation. I don't think there's any layering violation, it's just an id.
> 

This still sounds like a layering violation. I don't think protocols should have any knowledge of the web page period.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list