Commit 8bfd8394 authored by phoglund's avatar phoglund Committed by Commit bot

Fixing and re-enabling WebRTC/AppRTC integration test.

This patch needs to land after
https://codereview.chromium.org/872423002/ since the path
to the AppRTC code changes.

BUG=451985

Review URL: https://codereview.chromium.org/864973003

Cr-Commit-Position: refs/heads/master@{#313514}
parent e45ccdcc
...@@ -86,7 +86,8 @@ class WebRtcApprtcBrowserTest : public WebRtcTestBase { ...@@ -86,7 +86,8 @@ class WebRtcApprtcBrowserTest : public WebRtcTestBase {
} }
base::FilePath apprtc_dir = base::FilePath apprtc_dir =
GetSourceDir().Append(FILE_PATH_LITERAL("out/apprtc")); GetSourceDir().Append(FILE_PATH_LITERAL(
"out/webrtc-samples/samples/web/content/apprtc"));
if (!base::PathExists(apprtc_dir)) { if (!base::PathExists(apprtc_dir)) {
LOG(ERROR) << "Missing AppRTC code at " << LOG(ERROR) << "Missing AppRTC code at " <<
apprtc_dir.value() << ". " << kAdviseOnGclientSolution; apprtc_dir.value() << ". " << kAdviseOnGclientSolution;
...@@ -225,7 +226,7 @@ class WebRtcApprtcBrowserTest : public WebRtcTestBase { ...@@ -225,7 +226,7 @@ class WebRtcApprtcBrowserTest : public WebRtcTestBase {
base::Process collider_server_; base::Process collider_server_;
}; };
IN_PROC_BROWSER_TEST_F(WebRtcApprtcBrowserTest, DISABLED_MANUAL_WorksOnApprtc) { IN_PROC_BROWSER_TEST_F(WebRtcApprtcBrowserTest, MANUAL_WorksOnApprtc) {
// Disabled on Win XP: http://code.google.com/p/webrtc/issues/detail?id=2703. // Disabled on Win XP: http://code.google.com/p/webrtc/issues/detail?id=2703.
if (OnWinXp()) if (OnWinXp())
return; return;
...@@ -256,7 +257,7 @@ IN_PROC_BROWSER_TEST_F(WebRtcApprtcBrowserTest, DISABLED_MANUAL_WorksOnApprtc) { ...@@ -256,7 +257,7 @@ IN_PROC_BROWSER_TEST_F(WebRtcApprtcBrowserTest, DISABLED_MANUAL_WorksOnApprtc) {
} }
#if defined(OS_LINUX) #if defined(OS_LINUX)
#define MAYBE_MANUAL_FirefoxApprtcInteropTest DISABLED_MANUAL_FirefoxApprtcInteropTest #define MAYBE_MANUAL_FirefoxApprtcInteropTest MANUAL_FirefoxApprtcInteropTest
#else #else
// Not implemented yet on Windows and Mac. // Not implemented yet on Windows and Mac.
#define MAYBE_MANUAL_FirefoxApprtcInteropTest DISABLED_MANUAL_FirefoxApprtcInteropTest #define MAYBE_MANUAL_FirefoxApprtcInteropTest DISABLED_MANUAL_FirefoxApprtcInteropTest
......
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