Index: chrome/browser/media/chrome_webrtc_video_quality_browsertest.cc |
diff --git a/chrome/browser/media/chrome_webrtc_video_quality_browsertest.cc b/chrome/browser/media/chrome_webrtc_video_quality_browsertest.cc |
index 280a8c05f07e15974b0aae5a8f9dacb440731b11..edb42b04c4fe7e504bdb259b59144c7d71f3edc9 100644 |
--- a/chrome/browser/media/chrome_webrtc_video_quality_browsertest.cc |
+++ b/chrome/browser/media/chrome_webrtc_video_quality_browsertest.cc |
@@ -106,10 +106,7 @@ static const struct VideoQualityTestConfig { |
// * zxing (see the CPP version at https://code.google.com/p/zxing) |
// * ffmpeg 0.11.1 or compatible version (see http://www.ffmpeg.org) |
// |
-// The test case will launch a custom binary (peerconnection_server) which will |
-// allow two WebRTC clients to find each other. |
-// |
-// The test also runs several other custom binaries - rgba_to_i420 converter and |
+// The test runs several custom binaries - rgba_to_i420 converter and |
// frame_analyzer. Both tools can be found under third_party/webrtc/tools. The |
// test also runs a stand alone Python implementation of a WebSocket server |
// (pywebsocket) and a barcode_decoder script. |
@@ -123,7 +120,6 @@ class WebRtcVideoQualityBrowserTest : public WebRtcTestBase, |
} |
virtual void SetUpInProcessBrowserTestFixture() OVERRIDE { |
- test::PeerConnectionServerRunner::KillAllPeerConnectionServers(); |
DetectErrorsInJavaScript(); // Look for errors in our rather complex js. |
} |
@@ -298,7 +294,6 @@ class WebRtcVideoQualityBrowserTest : public WebRtcTestBase, |
} |
protected: |
- test::PeerConnectionServerRunner peerconnection_server_; |
VideoQualityTestConfig test_config_; |
private: |
@@ -339,7 +334,6 @@ IN_PROC_BROWSER_TEST_P(WebRtcVideoQualityBrowserTest, |
ASSERT_TRUE(HasAllRequiredResources()); |
ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); |
ASSERT_TRUE(StartPyWebSocketServer()); |
- ASSERT_TRUE(peerconnection_server_.Start()); |
content::WebContents* left_tab = |
OpenPageAndGetUserMediaInNewTabWithConstraints( |
@@ -350,7 +344,10 @@ IN_PROC_BROWSER_TEST_P(WebRtcVideoQualityBrowserTest, |
embedded_test_server()->GetURL(test_config_.capture_page), |
test_config_.constraints); |
- EstablishCall(left_tab, right_tab); |
+ SetupPeerconnectionWithLocalStream(left_tab); |
+ SetupPeerconnectionWithLocalStream(right_tab); |
+ |
+ NegotiateCall(left_tab, right_tab); |
// Poll slower here to avoid flooding the log with messages: capturing and |
// sending frames take quite a bit of time. |
@@ -361,8 +358,6 @@ IN_PROC_BROWSER_TEST_P(WebRtcVideoQualityBrowserTest, |
polling_interval_msec)); |
HangUp(left_tab); |
- WaitUntilHangupVerified(left_tab); |
- WaitUntilHangupVerified(right_tab); |
EXPECT_TRUE(test::PollingWaitUntil( |
"haveMoreFramesToSend()", "no-more-frames", right_tab, |
@@ -371,7 +366,6 @@ IN_PROC_BROWSER_TEST_P(WebRtcVideoQualityBrowserTest, |
// Shut everything down to avoid having the javascript race with the analysis |
// tools. For instance, dont have console log printouts interleave with the |
// RESULT lines from the analysis tools (crbug.com/323200). |
- ASSERT_TRUE(peerconnection_server_.Stop()); |
ASSERT_TRUE(ShutdownPyWebSocketServer()); |
chrome::CloseWebContents(browser(), left_tab, false); |