RESOLVED FIXED 159497
[WK2][Cocoa] Disable ResourceResponse lazy initialization 
https://bugs.webkit.org/show_bug.cgi?id=159497
Summary [WK2][Cocoa] Disable ResourceResponse lazy initialization
Chris Dumez
Reported 2016-07-06 16:38:39 PDT
Disable ResponseResponse lazy initialization for WebKit2 as we always end up initializing all the fields anyway when sending the response to the WebContent process via IPC. Even worse, we always do a partial lazy initialization (common fields only) before the IPC so we always do the initialization in 2 steps. This normally would not be an issue but the 2 steps actually have some overlap because both call CFHTTPMessageCopyAllHeaderFields(), which is expensive (See <rdar://problem/26796059>) to first get the common HTTP headers, then ALL the HTTP headers.
Attachments
Patch (4.76 KB, patch)
2016-07-06 16:54 PDT, Chris Dumez 		achristensen: review+
achristensen: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-07-06 16:39:26 PDT
<rdar://problem/27209066>
Chris Dumez
Comment 2 2016-07-06 16:54:52 PDT
Created attachment 282963 [details] Patch
Alex Christensen
Comment 3 2016-07-06 17:14:21 PDT
Comment on attachment 282963 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=282963&action=review > Source/WebCore/ChangeLog:3 > + [WK2][Cocoa] Disable ResponseResponse lazy initialization ResourceResponse.
Chris Dumez
Comment 4 2016-07-06 18:33:22 PDT
Committed r202888: <http://trac.webkit.org/changeset/202888>
Note You need to log in before you can comment on or make changes to this bug.