Commit f359bc16 authored by Dominic Battré's avatar Dominic Battré Committed by Commit Bot

Revert "Disable 4 tests in webrtc_browsertest.cc"

This reverts commit 5945cdc3.

Reason for revert: Disabling is non-actionable, see crbug.com/1010061#c11

Original change's description:
> Disable 4 tests in webrtc_browsertest.cc
> 
> TBR=hbos@chromium.org
> NOTRY=true
> 
> Bug: 1010061
> Change-Id: Ib35d542205f85f026a1f6df271dbba4c1b6683e6
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1833291
> Reviewed-by: Xida Chen <xidachen@chromium.org>
> Commit-Queue: Xida Chen <xidachen@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#701589}

TBR=hbos@chromium.org,xidachen@chromium.org

Change-Id: Icb2d8f84fd4fa1f1ecd6ae152f8b930949e019db
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1010061
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835354Reviewed-by: default avatarDominic Battré <battre@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#701923}
parent 4d26cf1b
......@@ -60,9 +60,7 @@ IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest, CanSetupAudioAndVideoCall) {
MakeTypicalPeerConnectionCall("call({video: true, audio: true});");
}
// crbug.com:1010061: disabled due to flakiness.
IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest,
DISABLED_NetworkProcessCrashRecovery) {
IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest, NetworkProcessCrashRecovery) {
if (!IsOutOfProcessNetworkService())
return;
MakeTypicalPeerConnectionCall("call({video: true, audio: true});");
......@@ -150,9 +148,8 @@ IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest, CanForwardRemoteStream) {
"callAndForwardRemoteStream({video: true, audio: true});");
}
// crbug.com:1010061: disabled due to flakiness.
IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest,
DISABLED_NoCrashWhenConnectChromiumSinkToRemoteTrack) {
NoCrashWhenConnectChromiumSinkToRemoteTrack) {
MakeTypicalPeerConnectionCall("ConnectChromiumSinkToRemoteAudioTrack();");
}
......@@ -161,9 +158,15 @@ IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest,
// video is playing for the call even if the initiating client don't support
// MSID. http://tools.ietf.org/html/draft-alvestrand-rtcweb-msid-02
// Fails with TSAN. https://crbug.com/756568
// crbug.com:1010061: disabled due to flakiness.
#if defined(THREAD_SANITIZER)
#define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle \
DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle
#else
#define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle \
CanSetupAudioAndVideoCallWithoutMsidAndBundle
#endif
IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest,
DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle) {
MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle) {
MakeTypicalPeerConnectionCall("callWithoutMsidAndBundle();");
}
......@@ -203,9 +206,7 @@ IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest, CallAndModifyStream) {
"callWithNewVideoMediaStreamLaterSwitchToAudio();");
}
// crbug.com:1010061: disabled due to flakiness.
IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest,
DISABLED_AddTwoMediaStreamsToOnePC) {
IN_PROC_BROWSER_TEST_F(MAYBE_WebRtcBrowserTest, AddTwoMediaStreamsToOnePC) {
MakeTypicalPeerConnectionCall("addTwoMediaStreamsToOneConnection();");
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment