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

Unified Diff: chrome/browser/media/chrome_webrtc_browsertest.cc

Issue 1762413002: Cleanup: remove prefix chrome_ of chrome/browser/media/chrome* files (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: reran the commands Created 4 years, 9 months 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
Index: chrome/browser/media/chrome_webrtc_browsertest.cc
diff --git a/chrome/browser/media/chrome_webrtc_browsertest.cc b/chrome/browser/media/chrome_webrtc_browsertest.cc
deleted file mode 100644
index 5a22b015a884c93a57256bdab3bd78f6460344bb..0000000000000000000000000000000000000000
--- a/chrome/browser/media/chrome_webrtc_browsertest.cc
+++ /dev/null
@@ -1,116 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/command_line.h"
-#include "base/strings/stringprintf.h"
-#include "build/build_config.h"
-#include "chrome/browser/media/webrtc_browsertest_base.h"
-#include "chrome/browser/media/webrtc_browsertest_common.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_tabstrip.h"
-#include "chrome/browser/ui/tabs/tab_strip_model.h"
-#include "chrome/common/chrome_switches.h"
-#include "chrome/test/base/in_process_browser_test.h"
-#include "chrome/test/base/ui_test_utils.h"
-#include "content/public/common/content_switches.h"
-#include "content/public/common/feature_h264_with_openh264_ffmpeg.h"
-#include "content/public/common/features.h"
-#include "content/public/test/browser_test_utils.h"
-#include "media/base/media_switches.h"
-#include "net/test/embedded_test_server/embedded_test_server.h"
-
-static const char kMainWebrtcTestHtmlPage[] =
- "/webrtc/webrtc_jsep01_test.html";
-
-// Top-level integration test for WebRTC. It always uses fake devices; see
-// WebRtcWebcamBrowserTest for a test that acquires any real webcam on the
-// system.
-class WebRtcBrowserTest : public WebRtcTestBase {
- public:
- void SetUpInProcessBrowserTestFixture() override {
- DetectErrorsInJavaScript(); // Look for errors in our rather complex js.
- }
-
- void SetUpCommandLine(base::CommandLine* command_line) override {
- // Ensure the infobar is enabled, since we expect that in this test.
- EXPECT_FALSE(command_line->HasSwitch(switches::kUseFakeUIForMediaStream));
-
- // Always use fake devices.
- command_line->AppendSwitch(switches::kUseFakeDeviceForMediaStream);
-
- // Flag used by TestWebAudioMediaStream to force garbage collection.
- command_line->AppendSwitchASCII(switches::kJavaScriptFlags, "--expose-gc");
- }
-
- void RunsAudioVideoWebRTCCallInTwoTabs(std::string video_codec) {
- if (OnWinXp()) return;
-
- ASSERT_TRUE(embedded_test_server()->Start());
-
- content::WebContents* left_tab =
- OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
- content::WebContents* right_tab =
- OpenTestPageAndGetUserMediaInNewTab(kMainWebrtcTestHtmlPage);
-
- SetupPeerconnectionWithLocalStream(left_tab);
- SetupPeerconnectionWithLocalStream(right_tab);
-
- NegotiateCall(left_tab, right_tab, video_codec);
-
- StartDetectingVideo(left_tab, "remote-view");
- StartDetectingVideo(right_tab, "remote-view");
-
-#if !defined(OS_MACOSX)
- // Video is choppy on Mac OS X. http://crbug.com/443542.
- WaitForVideoToPlay(left_tab);
- WaitForVideoToPlay(right_tab);
-#endif
-
- HangUp(left_tab);
- HangUp(right_tab);
- }
-};
-
-IN_PROC_BROWSER_TEST_F(WebRtcBrowserTest,
- RunsAudioVideoWebRTCCallInTwoTabsVP8) {
- RunsAudioVideoWebRTCCallInTwoTabs("VP8");
-}
-
-IN_PROC_BROWSER_TEST_F(WebRtcBrowserTest,
- RunsAudioVideoWebRTCCallInTwoTabsVP9) {
- RunsAudioVideoWebRTCCallInTwoTabs("VP9");
-}
-
-#if BUILDFLAG(RTC_USE_H264)
-
-IN_PROC_BROWSER_TEST_F(WebRtcBrowserTest,
- RunsAudioVideoWebRTCCallInTwoTabsH264) {
- // Only run test if run-time feature corresponding to |rtc_use_h264| is on.
- if (!base::FeatureList::IsEnabled(content::kWebRtcH264WithOpenH264FFmpeg)) {
- LOG(WARNING) << "Run-time feature WebRTC-H264WithOpenH264FFmpeg disabled. "
- "Skipping WebRtcBrowserTest.RunsAudioVideoWebRTCCallInTwoTabsH264 "
- "(test \"OK\")";
- return;
- }
- RunsAudioVideoWebRTCCallInTwoTabs("H264");
-}
-
-#endif // BUILDFLAG(RTC_USE_H264)
-
-IN_PROC_BROWSER_TEST_F(WebRtcBrowserTest, TestWebAudioMediaStream) {
- // This tests against crash regressions for the WebAudio-MediaStream
- // integration.
- if (OnWinXp()) return;
-
- ASSERT_TRUE(embedded_test_server()->Start());
- GURL url(embedded_test_server()->GetURL("/webrtc/webaudio_crash.html"));
- ui_test_utils::NavigateToURL(browser(), url);
- content::WebContents* tab =
- browser()->tab_strip_model()->GetActiveWebContents();
-
- // A sleep is necessary to be able to detect the crash.
- test::SleepInJavascript(tab, 1000);
-
- ASSERT_FALSE(tab->IsCrashed());
-}

Powered by Google App Engine
This is Rietveld 408576698