New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 879271 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Sep 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature

Blocking:
issue 878465



Sign in to add a comment

Support domain name ICE candidates in Chromium

Project Member Reported by zstein@chromium.org, Aug 30

Issue description

This is the Chromium companion bug to  http://crbug.com/webrtc/4165 .

WebRTC now supports domain names in ICE candidates, for example "your.hostname" in: a=candidate:1 1 udp 2130706431 your.hostname 11365 typ host generation 0.

Chromium needs to pass an AsyncResolverFactory to PeerConnection during creation in order for the name resolution to be triggered by WebRTC.

AsyncResolverFactory was introduced here: https://webrtc-review.googlesource.com/c/src/+/91300
 
Blocking: 878465
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c600823b01b687f788b2cf2ade1d55c7d1c41772

commit c600823b01b687f788b2cf2ade1d55c7d1c41772
Author: Zach Stein <zstein@chromium.org>
Date: Tue Sep 04 17:19:03 2018

Enable domain name ICE candidates in Chromium

Adds an AsyncResolverFactory to PeerConnections created by PeerConnectionDependencyFactory.
Along with webrtc-review.googlesource.com/85540, this allows ICE candidates hostnames to be resolved by WebRTC.
Also moves to the new, non-deprecated CreatePeerConnection call.

Bug:  chromium:879271 
Change-Id: I96e26d40fb7e0cfbd0eb1623bd00bb3241ca3c07
Reviewed-on: https://chromium-review.googlesource.com/1196170
Reviewed-by: Florent Castelli <orphis@chromium.org>
Commit-Queue: Zach Stein <zstein@google.com>
Cr-Commit-Position: refs/heads/master@{#588562}
[modify] https://crrev.com/c600823b01b687f788b2cf2ade1d55c7d1c41772/content/renderer/media/webrtc/peer_connection_dependency_factory.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified by replacing the IPv4 address in an ICE candidate with my machine's hostname using a modified version of the munge SDP sample (https://steinz.github.io/samples/src/content/peerconnection/munge-sdp/).

Sign in to add a comment