Chromium Code Reviews| Index: content/renderer/media/rtc_peer_connection_handler.cc |
| diff --git a/content/renderer/media/rtc_peer_connection_handler.cc b/content/renderer/media/rtc_peer_connection_handler.cc |
| index 3acd36a6a07ab41885013976a903549de403319c..50162fca16b26b3248c9e72e50b6031a04fecd1f 100644 |
| --- a/content/renderer/media/rtc_peer_connection_handler.cc |
| +++ b/content/renderer/media/rtc_peer_connection_handler.cc |
| @@ -243,7 +243,9 @@ void GetNativeRtcConfiguration( |
| webrtc::PeerConnectionInterface::IceServer server; |
| server.username = blink_server.username.utf8(); |
| server.password = blink_server.credential.utf8(); |
| - server.uri = blink_server.url.string().utf8(); |
| + for (const blink::WebURL& url : blink_server.urls) { |
| + server.urls.push_back(url.string().utf8()); |
| + } |
| webrtc_config->servers.push_back(server); |
| } |
| @@ -300,6 +302,75 @@ void GetNativeRtcConfiguration( |
| } |
| } |
| +void GetBlinkRtcConfiguration( |
| + const webrtc::PeerConnectionInterface::RTCConfiguration& webrtc_config, |
| + blink::WebRTCConfiguration* blink_config) { |
| + blink::WebVector<blink::WebRTCIceServer> iceServers( |
| + webrtc_config.servers.size()); |
| + for (size_t i = 0; i < webrtc_config.servers.size(); i++) { |
| + webrtc::PeerConnectionInterface::IceServer ice_server = |
|
Taylor_Brandstetter
2017/03/02 22:30:45
Can use a const ref here instead of making a copy.
zhihuang1
2017/03/09 19:13:31
Done.
|
| + webrtc_config.servers[i]; |
| + blink::WebRTCIceServer blink_server; |
| + blink::WebVector<blink::WebURL> blink_urls(ice_server.urls.size()); |
| + for (size_t j = 0; j < ice_server.urls.size(); j++) { |
| + blink_urls[j] = (GURL(ice_server.urls[j])); |
| + } |
| + blink_server.urls = std::move(blink_urls); |
| + blink_server.username = blink::WebString::fromUTF8(ice_server.username); |
| + blink_server.credential = blink::WebString::fromUTF8(ice_server.password); |
| + iceServers[i] = blink_server; |
| + } |
| + blink_config->iceServers = std::move(iceServers); |
| + |
| + switch (webrtc_config.type) { |
| + case webrtc::PeerConnectionInterface::kNone: |
| + blink_config->iceTransportPolicy = blink::WebRTCIceTransportPolicy::kNone; |
| + break; |
| + case webrtc::PeerConnectionInterface::kRelay: |
| + blink_config->iceTransportPolicy = |
| + blink::WebRTCIceTransportPolicy::kRelay; |
| + break; |
| + case webrtc::PeerConnectionInterface::kAll: |
| + blink_config->iceTransportPolicy = blink::WebRTCIceTransportPolicy::kAll; |
| + break; |
| + default: |
| + NOTREACHED(); |
| + } |
| + |
| + switch (webrtc_config.bundle_policy) { |
| + case webrtc::PeerConnectionInterface::kBundlePolicyBalanced: |
| + blink_config->bundlePolicy = blink::WebRTCBundlePolicy::kBalanced; |
| + break; |
| + case webrtc::PeerConnectionInterface::kBundlePolicyMaxBundle: |
| + blink_config->bundlePolicy = blink::WebRTCBundlePolicy::kMaxBundle; |
| + break; |
| + case webrtc::PeerConnectionInterface::kBundlePolicyMaxCompat: |
| + blink_config->bundlePolicy = blink::WebRTCBundlePolicy::kMaxCompat; |
| + break; |
| + default: |
| + NOTREACHED(); |
|
Taylor_Brandstetter
2017/03/02 22:30:45
nit: Don't need default case here.
zhihuang1
2017/03/09 19:13:31
Done.
|
| + } |
| + |
| + switch (webrtc_config.rtcp_mux_policy) { |
| + case webrtc::PeerConnectionInterface::kRtcpMuxPolicyNegotiate: |
| + blink_config->rtcpMuxPolicy = blink::WebRTCRtcpMuxPolicy::kNegotiate; |
| + break; |
| + case webrtc::PeerConnectionInterface::kRtcpMuxPolicyRequire: |
| + blink_config->rtcpMuxPolicy = blink::WebRTCRtcpMuxPolicy::kRequire; |
| + break; |
| + default: |
|
Taylor_Brandstetter
2017/03/02 22:30:45
nit: Don't need default case here.
zhihuang1
2017/03/09 19:13:33
Done.
|
| + NOTREACHED(); |
| + } |
| + |
| + blink::WebVector<std::unique_ptr<blink::WebRTCCertificate>> certificates( |
| + webrtc_config.certificates.size()); |
| + for (size_t i = 0; i < webrtc_config.certificates.size(); i++) { |
| + certificates[i] = std::unique_ptr<blink::WebRTCCertificate>( |
| + new RTCCertificate(webrtc_config.certificates[i])); |
| + } |
| + blink_config->certificates = std::move(certificates); |
| +} |
| + |
| void CopyConstraintsIntoRtcConfiguration( |
| const blink::WebMediaConstraints constraints, |
| webrtc::PeerConnectionInterface::RTCConfiguration* configuration) { |
| @@ -1455,6 +1526,15 @@ blink::WebRTCErrorType RTCPeerConnectionHandler::setConfiguration( |
| return blink_error.type(); |
| } |
| +blink::WebRTCConfiguration RTCPeerConnectionHandler::getConfiguration() { |
| + DCHECK(thread_checker_.CalledOnValidThread()); |
| + webrtc::PeerConnectionInterface::RTCConfiguration webrtc_config = |
| + native_peer_connection_->GetConfiguration(); |
| + blink::WebRTCConfiguration blink_config; |
| + GetBlinkRtcConfiguration(webrtc_config, &blink_config); |
| + return blink_config; |
| +} |
| + |
| bool RTCPeerConnectionHandler::addICECandidate( |
| const blink::WebRTCVoidRequest& request, |
| const blink::WebRTCICECandidate& candidate) { |