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

Unified Diff: media/gpu/BUILD.gn

Issue 2760513002: h264_decoder_unittests: Initial Change. (Closed)
Patch Set: Address kcwu's comments. Created 3 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: media/gpu/BUILD.gn
diff --git a/media/gpu/BUILD.gn b/media/gpu/BUILD.gn
index 2bd2fe0af0253f48b0c5aee6aea84c801dbd1f1d..e7e867ae31b3c0db6bc074cdb6d2525fdae4d5ad 100644
--- a/media/gpu/BUILD.gn
+++ b/media/gpu/BUILD.gn
@@ -128,6 +128,7 @@ component("gpu") {
defines = [ "MEDIA_GPU_IMPLEMENTATION" ]
sources = [
+ "accelerated_video_decoder.h",
"fake_video_decode_accelerator.cc",
"fake_video_decode_accelerator.h",
"gpu_video_accelerator_util.cc",
@@ -135,6 +136,10 @@ component("gpu") {
"gpu_video_decode_accelerator_factory.cc",
"gpu_video_decode_accelerator_factory.h",
"gpu_video_decode_accelerator_helpers.h",
+ "h264_decoder.cc",
+ "h264_decoder.h",
+ "h264_dpb.cc",
+ "h264_dpb.h",
"shared_memory_region.cc",
"shared_memory_region.h",
]
@@ -218,15 +223,6 @@ component("gpu") {
}
}
- if (is_chromeos || is_win) {
- sources += [
- "accelerated_video_decoder.h",
- "h264_decoder.cc",
- "h264_decoder.h",
- "h264_dpb.cc",
- "h264_dpb.h",
- ]
- }
if (is_chromeos) {
sources += [
"vp8_decoder.cc",
@@ -523,6 +519,29 @@ if (is_chromeos) {
}
}
+source_set("unit_tests") {
+ testonly = true
+ deps = [
+ "//base",
+ "//media/base:test_support",
+ "//media/gpu",
+ "//testing/gtest",
+ ]
+ sources = [
+ "h264_decoder_unittest.cc",
+ ]
+
+ if (is_android) {
+ deps += [
+ # The test needs the java dependencies to add the java classes for their
+ # native counterparts to the test apk.
+ ":android_video_decode_accelerator_unittests",
+ "//media/base/android:media_java",
+ "//ui/android:ui_java",
+ ]
+ }
+}
+
test("video_decode_accelerator_service_unittest") {
sources = [
"ipc/service/gpu_jpeg_decode_accelerator_unittest.cc",

Powered by Google App Engine
This is Rietveld 408576698