Bug 159497 - [WK2][Cocoa] Disable ResourceResponse lazy initialization
Summary: [WK2][Cocoa] Disable ResourceResponse lazy initialization
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
Keywords: InRadar
Depends on:
Reported: 2016-07-06 16:38 PDT by Chris Dumez
Modified: 2016-07-06 18:33 PDT (History)
4 users (show)

See Also:

Patch (4.76 KB, patch)
2016-07-06 16:54 PDT, Chris Dumez
achristensen: review+
achristensen: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 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.
Comment 1 Radar WebKit Bug Importer 2016-07-06 16:39:26 PDT
Comment 2 Chris Dumez 2016-07-06 16:54:52 PDT
Created attachment 282963 [details]
Comment 3 Alex Christensen 2016-07-06 17:14:21 PDT
Comment on attachment 282963 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=282963&action=review

> Source/WebCore/ChangeLog:3
> +        [WK2][Cocoa] Disable ResponseResponse lazy initialization

Comment 4 Chris Dumez 2016-07-06 18:33:22 PDT
Committed r202888: <http://trac.webkit.org/changeset/202888>