[Webkit-unassigned] [Bug 245576] [SOUP] Should not hit internal error preconnecting when preconnect is disabled

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 27 13:39:05 PDT 2022


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

--- Comment #4 from Alex Christensen <achristensen at apple.com> ---
Comment on attachment 462600
  --> https://bugs.webkit.org/attachment.cgi?id=462600
Add ENABLE(SERVER_PRECONNECT) guards

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

>> Source/WebCore/html/HTMLAnchorElement.cpp:542
>> +#endif
> 
> It would be better to add the guard around the block below. But I wonder if it would be better to check linkPreconnectEnabled in settings instead. Alex? Chris?

Yeah, this isn't great.  It adds a bunch of unreachable code when SERVER_PRECONNECT is disabled.

-- 
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/20220927/d5049f1b/attachment.htm>


More information about the webkit-unassigned mailing list