From 8444405aa4decdc56a5ab09c6eed57a6530cad2d Mon Sep 17 00:00:00 2001
From: sakal <sakal@webrtc.org>
Date: Wed, 15 Mar 2017 05:50:13 -0700
Subject: [PATCH] Try re-enabling two PeerConnectionClientTests:

- org.appspot.apprtc.test.PeerConnectionClientTest#testCaptureFormatChange
- org.appspot.apprtc.test.PeerConnectionClientTest#testVideoSourceRestart

The tests were previously flaky but significant changes to the test
architecture have been made since. Opportunistically try to re-enable
them.

BUG=webrtc:6478

Review-Url: https://codereview.webrtc.org/2748863003
Cr-Commit-Position: refs/heads/master@{#17245}
---
 .../appspot/apprtc/test/PeerConnectionClientTest.java    | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java b/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
index af91e08c7a..33956f77f2 100644
--- a/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
+++ b/webrtc/examples/androidtests/src/org/appspot/apprtc/test/PeerConnectionClientTest.java
@@ -15,7 +15,6 @@ import static org.junit.Assert.fail;
 
 import android.os.Build;
 import android.support.test.InstrumentationRegistry;
-import android.support.test.filters.FlakyTest;
 import android.support.test.filters.SmallTest;
 import android.util.Log;
 import java.util.LinkedList;
@@ -593,10 +592,8 @@ public class PeerConnectionClientTest implements PeerConnectionEvents {
 
   // Checks if video source can be restarted - simulate app goes to
   // background and back to foreground.
-  // Disabled because of https://bugs.chromium.org/p/webrtc/issues/detail?id=6478
   @Test
-  @FlakyTest
-  //@SmallTest
+  @SmallTest
   public void testVideoSourceRestart() throws InterruptedException {
     Log.d(TAG, "testVideoSourceRestart");
     loopback = true;
@@ -644,10 +641,8 @@ public class PeerConnectionClientTest implements PeerConnectionEvents {
   }
 
   // Checks if capture format can be changed on fly and decoder can be reset properly.
-  // Disabled because of https://bugs.chromium.org/p/webrtc/issues/detail?id=6478
   @Test
-  @FlakyTest
-  //@SmallTest
+  @SmallTest
   public void testCaptureFormatChange() throws InterruptedException {
     Log.d(TAG, "testCaptureFormatChange");
     loopback = true;