Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(88)

Unified Diff: third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html

Issue 2721163002: Add support for RTCConfiguration.iceCandidatePoolSize. (Closed)
Patch Set: Update layout test expectations due to switch from unsigned short to octet. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 8fca4d0672815000bc07cdfeff1271ef0a498c14..a8c2bac4eb2b0d765917bf80c2731f9e709e3a73 100644
--- a/third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html
+++ b/third_party/WebKit/LayoutTests/fast/peerconnection/RTCPeerConnection.html
@@ -45,6 +45,13 @@ shouldNotThrow("new RTCPeerConnection({iceServers:[], rtcpMuxPolicy:'negotiate'}
shouldNotThrow("new RTCPeerConnection({iceServers:[], rtcpMuxPolicy:'require'});");
shouldThrow("new RTCPeerConnection({iceServers:[], rtcpMuxPolicy:'foo'});");
+shouldNotThrow("new RTCPeerConnection({iceCandidatePoolSize:0});");
+shouldNotThrow("new RTCPeerConnection({iceCandidatePoolSize:1});");
+shouldNotThrow("new RTCPeerConnection({iceCandidatePoolSize:5});");
+shouldThrow("new RTCPeerConnection({iceCandidatePoolSize:-1});");
+shouldThrow("new RTCPeerConnection({iceCandidatePoolSize:99999999});");
foolip 2017/03/14 14:49:06 Can you also try 256?
Taylor_Brandstetter 2017/03/14 22:23:11 Done.
+shouldThrow("new RTCPeerConnection({iceCandidatePoolSize:'foo'});");
+
// Deprecated.
shouldNotThrow("new RTCPeerConnection(null, {mandatory:{valid_and_supported_1:1}});");
shouldNotThrow("new RTCPeerConnection(null, {mandatory:{valid_and_supported_1:1, valid_and_supported_2:1}});");

Powered by Google App Engine
This is Rietveld 408576698