| Index: trunk/src/chrome/browser/media/webrtc_browsertest_common.cc
|
| ===================================================================
|
| --- trunk/src/chrome/browser/media/webrtc_browsertest_common.cc (revision 260283)
|
| +++ trunk/src/chrome/browser/media/webrtc_browsertest_common.cc (working copy)
|
| @@ -15,75 +15,8 @@
|
| #include "chrome/browser/ui/browser_tabstrip.h"
|
| #include "content/public/test/browser_test_utils.h"
|
|
|
| -namespace test {
|
| -
|
| -const base::FilePath::CharType kReferenceVideosDirName[] =
|
| - FILE_PATH_LITERAL("webrtc.DEPS/webrtc_videos");
|
| -const base::FilePath::CharType kReferenceFileName360p[] =
|
| - FILE_PATH_LITERAL("reference_video_640x360_30fps");
|
| -const base::FilePath::CharType kYuvFileExtension[] = FILE_PATH_LITERAL("yuv");
|
| -const base::FilePath::CharType kY4mFileExtension[] = FILE_PATH_LITERAL("y4m");
|
| -
|
| -// This message describes how to modify your .gclient to get the reference
|
| -// video files downloaded for you.
|
| -static const char kAdviseOnGclientSolution[] =
|
| - "You need to add this solution to your .gclient to run this test:\n"
|
| - "{\n"
|
| - " \"name\" : \"webrtc.DEPS\",\n"
|
| - " \"url\" : \"svn://svn.chromium.org/chrome/trunk/deps/"
|
| - "third_party/webrtc/webrtc.DEPS\",\n"
|
| - "}";
|
| -
|
| const int kDefaultPollIntervalMsec = 250;
|
|
|
| -base::FilePath GetReferenceVideosDir() {
|
| - // FilePath does not tolerate relative paths, and we want to hang the
|
| - // kReferenceVideosDirName at the same level as Chromium codebase, so we
|
| - // need to subtract the trailing .../src manually from the path.
|
| - const size_t src_token_length = 3u;
|
| - const base::FilePath::StringType src_token(FILE_PATH_LITERAL("src"));
|
| -
|
| - base::FilePath source_dir;
|
| - PathService::Get(base::DIR_SOURCE_ROOT, &source_dir);
|
| -
|
| - base::FilePath::StringType path = source_dir.value();
|
| - DCHECK_GT(path.size(), src_token_length);
|
| - std::size_t found = path.rfind(src_token);
|
| - if (found != std::string::npos)
|
| - path.erase(found, src_token_length);
|
| - return base::FilePath(path).Append(kReferenceVideosDirName);
|
| -}
|
| -
|
| -bool HasReferenceFilesInCheckout() {
|
| - if (!base::PathExists(GetReferenceVideosDir())) {
|
| - LOG(ERROR)
|
| - << "Cannot find the working directory for the reference video "
|
| - << "files, expected at " << GetReferenceVideosDir().value() << ". " <<
|
| - kAdviseOnGclientSolution;
|
| - return false;
|
| - }
|
| - base::FilePath webrtc_reference_video_yuv = GetReferenceVideosDir()
|
| - .Append(kReferenceFileName360p).AddExtension(kYuvFileExtension);
|
| - if (!base::PathExists(webrtc_reference_video_yuv)) {
|
| - LOG(ERROR)
|
| - << "Missing YUV reference video to be used for quality"
|
| - << " comparison, expected at " << webrtc_reference_video_yuv.value()
|
| - << ". " << kAdviseOnGclientSolution;
|
| - return false;
|
| - }
|
| -
|
| - base::FilePath webrtc_reference_video_y4m = GetReferenceVideosDir()
|
| - .Append(kReferenceFileName360p).AddExtension(kY4mFileExtension);
|
| - if (!base::PathExists(webrtc_reference_video_y4m)) {
|
| - LOG(ERROR)
|
| - << "Missing Y4M reference video to be used for quality"
|
| - << " comparison, expected at "<< webrtc_reference_video_y4m.value()
|
| - << ". " << kAdviseOnGclientSolution;
|
| - return false;
|
| - }
|
| - return true;
|
| -}
|
| -
|
| bool SleepInJavascript(content::WebContents* tab_contents, int timeout_msec) {
|
| const std::string javascript = base::StringPrintf(
|
| "setTimeout(function() {"
|
| @@ -129,10 +62,9 @@
|
| LOG(ERROR) << "Failed to sleep.";
|
| }
|
| }
|
| - LOG(ERROR)
|
| - << "Timed out while waiting for " << javascript
|
| - << " to evaluate to " << evaluates_to << "; last result was '" << result
|
| - << "'";
|
| + LOG(ERROR) << "Timed out while waiting for " << javascript <<
|
| + " to evaluate to " << evaluates_to << "; last result was '" << result <<
|
| + "'";
|
| return false;
|
| }
|
|
|
| @@ -154,9 +86,8 @@
|
| peerconnection_server = peerconnection_server.Append(kServerExecutable);
|
|
|
| if (!base::PathExists(peerconnection_server)) {
|
| - LOG(ERROR)
|
| - << "Missing " << kServerExecutable << ". You must build "
|
| - << "it so it ends up next to the browser test binary.";
|
| + LOG(ERROR) << "Missing " << kServerExecutable << ". You must build "
|
| + "it so it ends up next to the browser test binary.";
|
| return false;
|
| }
|
|
|
| @@ -172,7 +103,7 @@
|
| return base::KillProcess(server_pid_, 0, false);
|
| }
|
|
|
| -void PeerConnectionServerRunner::KillAllPeerConnectionServers() {
|
| +void PeerConnectionServerRunner::KillAllPeerConnectionServersOnCurrentSystem() {
|
| if (!base::KillProcesses(kServerExecutable, -1, NULL)) {
|
| LOG(ERROR) << "Failed to kill instances of " << kServerExecutable << ".";
|
| return;
|
| @@ -180,5 +111,3 @@
|
| base::WaitForProcessesToExit(kServerExecutable,
|
| base::TimeDelta::FromSeconds(5), NULL);
|
| }
|
| -
|
| -} // namespace test
|
|
|