From 8ee1e5e6e63234df754fc5da453f88ca40307d6c Mon Sep 17 00:00:00 2001 From: Steve Anton Date: Tue, 20 Feb 2018 17:19:19 -0800 Subject: [PATCH] Enable GetRemoteAudioSSLCertificate tests for Unified Plan They were disabled since GetRemoteAudioSSLCertificate was written in terms of voice/video channel, which were not methods supported with Unified Plan. Now GetRemoteAudioSSLCertificate has been rewritten to work with RtpTransceivers, so the test can be enabled. Bug: webrtc:8764 Change-Id: I08b5fbcc0d69f36113a281c902db6508fa48ebdd Reviewed-on: https://webrtc-review.googlesource.com/55923 Reviewed-by: Seth Hampson Commit-Queue: Steve Anton Cr-Commit-Position: refs/heads/master@{#22115} --- pc/peerconnection_integrationtest.cc | 4 ---- 1 file changed, 4 deletions(-) diff --git a/pc/peerconnection_integrationtest.cc b/pc/peerconnection_integrationtest.cc index d5037a82b1..312407cbad 100644 --- a/pc/peerconnection_integrationtest.cc +++ b/pc/peerconnection_integrationtest.cc @@ -1591,10 +1591,6 @@ TEST_P(PeerConnectionIntegrationTest, EndToEndCallWithSdes) { // certificate once the DTLS handshake has finished. TEST_P(PeerConnectionIntegrationTest, GetRemoteAudioSSLCertificateReturnsExchangedCertificate) { - // TODO(bugs.webrtc.org/8764): Enable for Unified Plan once stats work. - if (sdp_semantics_ == SdpSemantics::kUnifiedPlan) { - return; - } auto GetRemoteAudioSSLCertificate = [](PeerConnectionWrapper* wrapper) { auto pci = reinterpret_cast(wrapper->pc()); auto pc = reinterpret_cast(pci->internal());