[Webkit-unassigned] [Bug 136701] NetworkResourceLoader cleanups
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Sep 10 09:58:45 PDT 2014
https://bugs.webkit.org/show_bug.cgi?id=136701
--- Comment #5 from Alexey Proskuryakov <ap at webkit.org> 2014-09-10 09:58:48 PST ---
(From update of attachment 237890)
View in context: https://bugs.webkit.org/attachment.cgi?id=237890&action=review
> Source/WebKit2/ChangeLog:21
> + Instead of copying everything out from NetworkResourceLoadParameters just include the whole thing as a const field.
I guess it's OK to try how this feels, but this is not how other "parameters" classes work, and it's really a layering violation between IPC and loading logic to do this.
> Source/WebKit2/ChangeLog:64
> + Use original request instead of the current one. This might fix a bug where we didn't use file backing over redirects.
I'd love to hear Brady's take on this. To me, this sounds incorrect - caching is always about the final URL, because redirect chains can and do change.
> Source/WebKit2/ChangeLog:69
> + Decode to SanboxExtensions to the actual type rather than a handle.
Good stuff - the complications with sandbox extensions were caused by lack of C++11 move support at the time the code was written, and the class itself wasn't RefCounted if I remember correctly.
--
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