[webkit-changes] [WebKit/WebKit] 5a0dc0: Remove duplicate DNS lookup implementation

Alex Christensen noreply at github.com
Fri May 26 09:04:40 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5a0dc0c7e3a5b1d90ad25e7901711012c173e4e4
      https://github.com/WebKit/WebKit/commit/5a0dc0c7e3a5b1d90ad25e7901711012c173e4e4
  Author: Alex Christensen <achristensen at apple.com>
  Date:   2023-05-26 (Fri, 26 May 2023)

  Changed paths:
    M Source/WebCore/platform/network/DNS.cpp
    M Source/WebCore/platform/network/DNS.h
    M Source/WebCore/platform/network/DNSResolveQueue.h
    M Source/WebCore/platform/network/cf/DNSResolveQueueCFNet.cpp
    M Source/WebCore/platform/network/cf/DNSResolveQueueCFNet.h
    M Source/WebKit/NetworkProcess/webrtc/NetworkRTCProvider.cpp
    M Source/WebKit/NetworkProcess/webrtc/NetworkRTCProvider.h
    R Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolver.cpp
    R Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolver.h
    R Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolverCocoa.cpp
    R Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolverCocoa.h
    M Source/WebKit/SourcesCocoa.txt
    M Source/WebKit/WebKit.xcodeproj/project.pbxproj

  Log Message:
  -----------
  Remove duplicate DNS lookup implementation
https://bugs.webkit.org/show_bug.cgi?id=257303
rdar://109812699

Reviewed by Youenn Fablet.

DNSServiceGetAddrInfo is the replacement for CFHostStartInfoResolution.

Remove NetworkRTCResolver in favor of implementing DNSResolveQueueCFNet::resolve
like the libSoup ports and sharing 1 implementation instead of having duplicate code.

* Source/WebCore/platform/network/DNS.cpp:
(WebCore::resolveDNS):
* Source/WebCore/platform/network/DNS.h:
(WebCore::IPAddress::fromSockAddrIn6): Deleted.
* Source/WebCore/platform/network/DNSResolveQueue.h:
* Source/WebCore/platform/network/cf/DNSResolveQueueCFNet.cpp:
(WebCore::DNSResolveQueueCFNet::CompletionHandlerWrapper::create):
(WebCore::DNSResolveQueueCFNet::CompletionHandlerWrapper::complete):
(WebCore::DNSResolveQueueCFNet::CompletionHandlerWrapper::addIPAddress):
(WebCore::DNSResolveQueueCFNet::CompletionHandlerWrapper::CompletionHandlerWrapper):
(WebCore::extractIPAddress):
(WebCore::dnsLookupCallback):
(WebCore::DNSResolveQueueCFNet::performDNSLookup):
(WebCore::DNSResolveQueueCFNet::platformResolve):
(WebCore::DNSResolveQueueCFNet::resolve):
(WebCore::DNSResolveQueueCFNet::stopResolve):
(WebCore::clientCallback): Deleted.
* Source/WebCore/platform/network/cf/DNSResolveQueueCFNet.h:
* Source/WebKit/NetworkProcess/webrtc/NetworkRTCProvider.cpp:
(WebKit::NetworkRTCProvider::close):
(WebKit::NetworkRTCProvider::createResolver):
(WebKit::NetworkRTCProvider::stopResolver):
* Source/WebKit/NetworkProcess/webrtc/NetworkRTCProvider.h:
* Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolver.cpp: Removed.
* Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolver.h: Removed.
* Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolverCocoa.cpp: Removed.
* Source/WebKit/NetworkProcess/webrtc/NetworkRTCResolverCocoa.h: Removed.
* Source/WebKit/SourcesCocoa.txt:
* Source/WebKit/WebKit.xcodeproj/project.pbxproj:

Canonical link: https://commits.webkit.org/264586@main




More information about the webkit-changes mailing list