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

Unified Diff: webrtc/api/peerconnection_unittest.cc

Issue 2546913003: Disable P2PTestConductor.LocalP2PTestDtlsBundleInIceRestart under msan (Closed)
Patch Set: Created 4 years 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/peerconnection_unittest.cc
diff --git a/webrtc/api/peerconnection_unittest.cc b/webrtc/api/peerconnection_unittest.cc
index 0ae10048818e0a732c2c6b2000fa78b3d9bd1192..4cb6190c9135446b47db3286032884ecc546b968 100644
--- a/webrtc/api/peerconnection_unittest.cc
+++ b/webrtc/api/peerconnection_unittest.cc
@@ -1721,7 +1721,16 @@ TEST_F(P2PTestConductor, LocalP2PTestDtlsTransferCallee) {
// This test sets up a non-bundle call and apply bundle during ICE restart. When
// bundle is in effect in the restart, the channel can successfully reset its
// DTLS-SRTP context.
-TEST_F(P2PTestConductor, LocalP2PTestDtlsBundleInIceRestart) {
+#if defined(MEMORY_SANITIZER)
+// Fails under MemorySanitizer:
+// See https://bugs.chromium.org/p/webrtc/issues/detail?id=6811
+#define MAYBE_LocalP2PTestDtlsBundleInIceRestart \
+ DISABLED_LocalP2PTestDtlsBundleInIceRestart
+#else
+#define MAYBE_LocalP2PTestDtlsBundleInIceRestart \
+ LocalP2PTestDtlsBundleInIceRestart
+#endif
+TEST_F(P2PTestConductor, MAYBE_LocalP2PTestDtlsBundleInIceRestart) {
MAYBE_SKIP_TEST(rtc::SSLStreamAdapter::HaveDtlsSrtp);
FakeConstraints setup_constraints;
setup_constraints.AddMandatory(MediaConstraintsInterface::kEnableDtlsSrtp,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698