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

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

Issue 247723006: Add HD browser test to WebRTC suite. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix Win compile error Created 6 years, 8 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/webrtc_browsertest_common.cc
diff --git a/chrome/browser/media/webrtc_browsertest_common.cc b/chrome/browser/media/webrtc_browsertest_common.cc
index 8eae1f8ad9cb5bdb41e35a25aeb89fc6e180c622..eb2e0ce87fddcff7f1389078c44d64e365a20261 100644
--- a/chrome/browser/media/webrtc_browsertest_common.cc
+++ b/chrome/browser/media/webrtc_browsertest_common.cc
@@ -23,6 +23,8 @@ const base::FilePath::CharType kReferenceFilesDirName[] =
FILE_PATH_LITERAL("webrtc/resources");
const base::FilePath::CharType kReferenceFileName360p[] =
FILE_PATH_LITERAL("reference_video_640x360_30fps");
+const base::FilePath::CharType kReferenceFileName720p[] =
+ FILE_PATH_LITERAL("reference_video_1280x720_30fps");
const base::FilePath::CharType kYuvFileExtension[] = FILE_PATH_LITERAL("yuv");
const base::FilePath::CharType kY4mFileExtension[] = FILE_PATH_LITERAL("y4m");
@@ -53,8 +55,13 @@ bool HasReferenceFilesInCheckout() {
kAdviseOnGclientSolution;
return false;
}
+ return HasYuvAndY4mFile(test::kReferenceFileName360p) &&
+ HasYuvAndY4mFile(test::kReferenceFileName720p);
+}
+
+bool HasYuvAndY4mFile(const base::FilePath::CharType* reference_file) {
base::FilePath webrtc_reference_video_yuv = GetReferenceFilesDir()
- .Append(kReferenceFileName360p).AddExtension(kYuvFileExtension);
+ .Append(reference_file).AddExtension(kYuvFileExtension);
if (!base::PathExists(webrtc_reference_video_yuv)) {
LOG(ERROR)
<< "Missing YUV reference video to be used for quality"
@@ -64,7 +71,7 @@ bool HasReferenceFilesInCheckout() {
}
base::FilePath webrtc_reference_video_y4m = GetReferenceFilesDir()
- .Append(kReferenceFileName360p).AddExtension(kY4mFileExtension);
+ .Append(reference_file).AddExtension(kY4mFileExtension);
if (!base::PathExists(webrtc_reference_video_y4m)) {
LOG(ERROR)
<< "Missing Y4M reference video to be used for quality"
« no previous file with comments | « chrome/browser/media/webrtc_browsertest_common.h ('k') | chrome/test/data/webrtc/webrtc_video_quality_test_hd.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698