Remove RTCIceTransportPolicy "none".

third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html

Index: third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html
diff --git a/third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html b/third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html
index ca6e780854dab4994b67dbc139e99155d9cebb37..d8976863dbb15b850ff13149937b1c4cabcd7be0 100644
--- a/third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html
+++ b/third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html
@@ -31,9 +31,15 @@ shouldThrow("new RTCPeerConnection({iceServers:[{url:'foo'}]});");
 shouldThrow("new RTCPeerConnection({iceServers:[{urls:'unsupported:scheme'}]});");
 shouldThrow("new RTCPeerConnection({iceServers:[{urls:[1, 'turn:foo.com']}]});");
 
-shouldNotThrow("new RTCPeerConnection({iceServers:[], iceTransports:'none'});");
+shouldNotThrow("new RTCPeerConnection({iceServers:[], iceTransportPolicy:'relay'});");

[foolip, 2017/04/21 16:14:29]

This is fine, but over time I think LayoutTests/fast/peerconnection/ must be
reconciled with LayoutTests/external/wpt/webrtc/, so making changes just to wpt
and where possible removing duplicate coverage would be my strategy.

+shouldNotThrow("new RTCPeerConnection({iceServers:[], iceTransportPolicy:'all'});");
+shouldThrow("new RTCPeerConnection({iceServers:[], iceTransportPolicy:'none'});");
+shouldThrow("new RTCPeerConnection({iceServers:[], iceTransportPolicy:'foo'});");
+
+// TODO(foolip): |iceTransports| is not in the spec.
 shouldNotThrow("new RTCPeerConnection({iceServers:[], iceTransports:'relay'});");
 shouldNotThrow("new RTCPeerConnection({iceServers:[], iceTransports:'all'});");
+shouldThrow("new RTCPeerConnection({iceServers:[], iceTransports:'none'});");
 shouldThrow("new RTCPeerConnection({iceServers:[], iceTransports:'foo'});");
 
 shouldNotThrow("new RTCPeerConnection({iceServers:[], bundlePolicy:'balanced'});");