RESOLVED FIXED 190681
Handle MDNS resolution of candidates through libwebrtc directly 
https://bugs.webkit.org/show_bug.cgi?id=190681
Summary Handle MDNS resolution of candidates through libwebrtc directly
youenn fablet
Reported 2018-10-17 12:48:39 PDT
Libwebrtc has proper support for MDNS name resolution. Let's use that.
Attachments
Patch (31.73 KB, patch)
2018-10-17 13:49 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Patch (34.42 KB, patch)
2018-10-29 07:37 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2018-10-17 13:49:17 PDT
Created attachment 352633 [details] Patch
WebKit Commit Bot
Comment 2 2018-10-18 11:15:26 PDT
Comment on attachment 352633 [details] Patch Clearing flags on attachment: 352633 Committed r237261: <https://trac.webkit.org/changeset/237261>
WebKit Commit Bot
Comment 3 2018-10-18 11:15:27 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 4 2018-10-18 11:16:24 PDT
<rdar://problem/45377118>
Ryan Haddad
Comment 5 2018-10-23 14:07:20 PDT
Reverted r237261 for reason: The layout test for this change crashes under GuardMalloc. Committed r237367: <https://trac.webkit.org/changeset/237367>
Ryan Haddad
Comment 6 2018-10-23 14:07:53 PDT
(In reply to Ryan Haddad from comment #5) > Reverted r237261 for reason: > > The layout test for this change crashes under GuardMalloc. > > Committed r237367: <https://trac.webkit.org/changeset/237367> Details in radar.
youenn fablet
Comment 7 2018-10-29 07:37:01 PDT
Created attachment 353288 [details] Patch
Eric Carlson
Comment 8 2018-10-29 11:31:04 PDT
Comment on attachment 353288 [details] Patch still looks good to me
WebKit Commit Bot
Comment 9 2018-10-29 14:31:24 PDT
Comment on attachment 353288 [details] Patch Clearing flags on attachment: 353288 Committed r237568: <https://trac.webkit.org/changeset/237568>
WebKit Commit Bot
Comment 10 2018-10-29 14:31:25 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.