Attachment 368325 Details for Bug 197315 – Patch

[patch] Patch

bug-197315-20190426095449.patch (text/plain), 3.04 KB, created by youenn fablet on 2019-04-26 09:54:50 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2019-04-26 09:54:50 PDT

Size: 3.04 KB

patch

obsolete

>Subversion Revision: 244579
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 3a279d5176796db9b95714209cafa0b63cf5f5d2..e819ccec6ae932bc897011ebc0d2093e3ab41086 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-04-26  Youenn Fablet  <youenn@apple.com>
>+
>+        Remove no longer needed mDNS ICE candidate resolution code
>+        https://bugs.webkit.org/show_bug.cgi?id=197315
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        No change of behavior, removing no longer needed code.
>+
>+        * Modules/mediastream/PeerConnectionBackend.cpp:
>+        (WebCore::PeerConnectionBackend::addIceCandidateSucceeded):
>+        (WebCore::PeerConnectionBackend::addIceCandidateFailed):
>+        * Modules/mediastream/PeerConnectionBackend.h:
>+
> 2019-04-25  Youenn Fablet  <youenn@apple.com>
> 
>         [Mac WK2 iOS Sim] Layout Test imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https.html is a flaky failure
>diff --git a/Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp b/Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp
>index 12a23e9072538e6cbd6652408c9470c896237c70..04ec535064a7c89591afbba2620577165814fc4d 100644
>--- a/Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp
>+++ b/Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp
>@@ -308,14 +308,10 @@ void PeerConnectionBackend::addIceCandidateSucceeded()
>     if (m_peerConnection.isClosed())
>         return;
> 
>-    // FIXME: Update remote description and set ICE connection state to checking if not already done so.
>     ASSERT(m_addIceCandidatePromise);
> 
>     m_addIceCandidatePromise->resolve();
>     m_addIceCandidatePromise = WTF::nullopt;
>-
>-    if (!m_waitingForMDNSResolution && m_finishedReceivingCandidates)
>-        endOfIceCandidates(WTFMove(*m_endOfIceCandidatePromise));
> }
> 
> void PeerConnectionBackend::addIceCandidateFailed(Exception&& exception)
>@@ -330,9 +326,6 @@ void PeerConnectionBackend::addIceCandidateFailed(Exception&& exception)
> 
>     m_addIceCandidatePromise->reject(WTFMove(exception));
>     m_addIceCandidatePromise = WTF::nullopt;
>-
>-    if (!m_waitingForMDNSResolution && m_finishedReceivingCandidates)
>-        endOfIceCandidates(WTFMove(*m_endOfIceCandidatePromise));
> }
> 
> void PeerConnectionBackend::fireICECandidateEvent(RefPtr<RTCIceCandidate>&& candidate, String&& serverURL)
>diff --git a/Source/WebCore/Modules/mediastream/PeerConnectionBackend.h b/Source/WebCore/Modules/mediastream/PeerConnectionBackend.h
>index d496ce1e7d5d8b36b25d5d8414a62e7a2a2044b9..4e28288910580b97cc1023079e05ecc414699bc5 100644
>--- a/Source/WebCore/Modules/mediastream/PeerConnectionBackend.h
>+++ b/Source/WebCore/Modules/mediastream/PeerConnectionBackend.h
>@@ -229,11 +229,6 @@ private:
>     bool m_negotiationNeeded { false };
>     bool m_finishedGatheringCandidates { false };
>     uint64_t m_waitingForMDNSRegistration { 0 };
>-
>-    bool m_finishedReceivingCandidates { false };
>-    uint64_t m_waitingForMDNSResolution { 0 };
>-
>-    HashMap<String, String> m_mdnsMapping;
> };
> 
> } // namespace WebCore

Actions: View | Formatted Diff | Diff

Attachments on bug 197315: 368325 | 368328 | 368359