Use Init dictionaries in RTCPeerConnection legacy methods.

third_party/WebKit/LayoutTests/fast/mediastream/RTCPeerConnection-ice.html

Index: third_party/WebKit/LayoutTests/fast/mediastream/RTCPeerConnection-ice.html
diff --git a/third_party/WebKit/LayoutTests/fast/mediastream/RTCPeerConnection-ice.html b/third_party/WebKit/LayoutTests/fast/mediastream/RTCPeerConnection-ice.html
index dff8fd217b2b3a3e8d2c5db6ae4c3ca86585fa20..cfa24e18a7eb93fc3d4cc1285ed4b5f90c2e6c6f 100644
--- a/third_party/WebKit/LayoutTests/fast/mediastream/RTCPeerConnection-ice.html
+++ b/third_party/WebKit/LayoutTests/fast/mediastream/RTCPeerConnection-ice.html
@@ -18,9 +18,27 @@ function onIceChange2()
     }
 }
 
-function addIceCandidateSuccess()
+function addIceCandidateSuccess1()

[hta - Chromium, 2016/05/26 04:56:41]

Is it time to rewrite this test as a testharness.js test?

[Guido Urdaneta, 2016/05/26 14:37:36]

Created a bug to migrate all mediastream tests and added TODO here.

 {
-    testPassed("addIceCandidateSuccess was called.");
+    testPassed("addIceCandidateSuccess1 was called.");
+    shouldNotThrow('pc.addIceCandidate({candidate: "candidate", sdpMid: 0}, addIceCandidateSuccess2, unexpectedCallback);');
+}
+
+function addIceCandidateSuccess2()
+{
+    testPassed("addIceCandidateSuccess2 was called.");
+    shouldNotThrow('pc.addIceCandidate({candidate: "candidate", sdpMLineIndex: 0}, addIceCandidateSuccess3, unexpectedCallback);');
+}
+
+function addIceCandidateSuccess3()
+{
+    testPassed("addIceCandidateSuccess3 was called.");
+    shouldNotThrow('pc.addIceCandidate({candidate: "candidate", sdpMid: 0, sdpMLineIndex: 0}, addIceCandidateSuccess4, unexpectedCallback);');
+}
+
+function addIceCandidateSuccess4()
+{
+    testPassed("addIceCandidateSuccess4 was called.");
     pc.oniceconnectionstatechange = onIceChange2;
     pc.close();
 }
@@ -43,10 +61,11 @@ function onIceChange1()
     if (pc.iceConnectionState === "completed") {
         testPassed("iceConnectionState is completed");
         iceCandidate = new RTCIceCandidate({candidate:"nano nano"});
-        shouldNotThrow('pc.addIceCandidate(null, addIceCandidateSuccess, unexpectedCallback).catch(expectedTypeError);');
+        shouldNotThrow('pc.addIceCandidate(null, unexpectedCallback, unexpectedCallback).catch(expectedTypeError);');
+        shouldNotThrow('pc.addIceCandidate({candidate: "candidate"}, unexpectedCallback, unexpectedCallback).catch(expectedTypeError);');
         shouldNotThrow('pc.addIceCandidate(iceCandidate, null, unexpectedCallback).catch(expectedTypeError);');
-        shouldNotThrow('pc.addIceCandidate(iceCandidate, addIceCandidateSuccess, null).catch(expectedTypeError);');
-        shouldNotThrow('pc.addIceCandidate(iceCandidate, addIceCandidateSuccess, unexpectedCallback);');
+        shouldNotThrow('pc.addIceCandidate(iceCandidate, unexpectedCallback, null).catch(expectedTypeError);');
+        shouldNotThrow('pc.addIceCandidate(iceCandidate, addIceCandidateSuccess1, unexpectedCallback);');
     }
 }