Create the API that returns the RTCConfiguration of the PeerConnection.

third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp

Index: third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
diff --git a/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp b/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
index 3ea3bc9602f0858f6c6515a79691b81cefd55fb2..70a2f93df40329a139e6838f861dc26acd8d6b64 100644
--- a/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
+++ b/third_party/WebKit/Source/modules/peerconnection/RTCPeerConnection.cpp
@@ -298,6 +298,7 @@ WebRTCConfiguration parseConfiguration(ExecutionContext* context,
       String username = iceServer.username();
       String credential = iceServer.credential();
 
+      Vector<WebURL> urls;
       for (const String& urlString : urlStrings) {
         KURL url(KURL(), urlString);
         if (!url.isValid()) {
@@ -320,8 +321,9 @@ WebRTCConfiguration parseConfiguration(ExecutionContext* context,
                                            "required when the URL scheme is "
                                            "\"turn\" or \"turns\".");
         }
-        iceServers.push_back(WebRTCIceServer{url, username, credential});
+        urls.push_back(url);
       }
+      iceServers.push_back(WebRTCIceServer{urls, username, credential});
     }
     webConfiguration.iceServers = iceServers;
   }
@@ -796,6 +798,64 @@ RTCSessionDescription* RTCPeerConnection::remoteDescription() {
   return RTCSessionDescription::create(webSessionDescription);
 }
 
+void RTCPeerConnection::getConfiguration(RTCConfiguration& rtcConfig) {
+  blink::WebRTCConfiguration blinkConfig = m_peerHandler->getConfiguration();
+
+  switch (blinkConfig.iceTransportPolicy) {
+    case blink::WebRTCIceTransportPolicy::kNone:
+      rtcConfig.setIceTransportPolicy("none");
+      break;
+    case blink::WebRTCIceTransportPolicy::kRelay:
+      rtcConfig.setIceTransportPolicy("relay");
+      break;
+    case blink::WebRTCIceTransportPolicy::kAll:
+      rtcConfig.setIceTransportPolicy("all");
+      break;
+  }
+
+  switch (blinkConfig.bundlePolicy) {
+    case blink::WebRTCBundlePolicy::kBalanced:
+      rtcConfig.setBundlePolicy("balanced");
+      break;
+    case blink::WebRTCBundlePolicy::kMaxBundle:
+      rtcConfig.setBundlePolicy("max-bundle");
+      break;
+    case blink::WebRTCBundlePolicy::kMaxCompat:
+      rtcConfig.setBundlePolicy("max-compat");
+      break;
+  }
+
+  switch (blinkConfig.rtcpMuxPolicy) {
+    case blink::WebRTCRtcpMuxPolicy::kNegotiate:
+      rtcConfig.setRtcpMuxPolicy("negotiate");
+      break;
+    case blink::WebRTCRtcpMuxPolicy::kRequire:
+      rtcConfig.setRtcpMuxPolicy("require");
+      break;
+  }
+
+  HeapVector<RTCIceServer> servers;
+  for (const WebRTCIceServer& blink_server : blinkConfig.iceServers) {
+    RTCIceServer server;
+    Vector<String> urlStrings;
+    StringOrStringSequence urls;
+    for (const WebURL& blink_url : blink_server.urls)
+      urlStrings.push_back(blink_url.string());
+    urls.setStringSequence(urlStrings);

[foolip, 2017/03/21 08:37:14]

There's some normalization here that isn't spelled out in the spec. The
configuration `{ "iceServers": [{ "urls": "stuns:stun.example.org" }] }` would
become `{ "iceServers": [{ "urls": ["stuns:stun.example.org"] }] }` (array of 1
URL) and actually so would `{ "iceServers": [{ "url": "stuns:stun.example.org"
}] }` (url in singular, not plural).

Can you update the spec to clearly define what should happen here and make sure
the tests cover that?

+    server.setURLs(urls);
+    server.setUsername(blink_server.username);
+    server.setCredential(blink_server.credential);
+    servers.push_back(server);
+  }
+  rtcConfig.setIceServers(servers);
+
+  HeapVector<Member<RTCCertificate>> certificates;
+  for (auto& blink_certificate : blinkConfig.certificates) {
+    certificates.push_back(new RTCCertificate(std::move(blink_certificate)));
+  }
+  rtcConfig.setCertificates(certificates);
+}
+
 void RTCPeerConnection::setConfiguration(
     ScriptState* scriptState,
     const RTCConfiguration& rtcConfiguration,