[media] Enable media_unittests for all platforms

Enable `media_unittests` for Cobalt, and list three tasks for the following unit_tests: - StarboardRenderer: b/379934658 - BidirectionalFitReuseAllocator: b/379934533 - starboard_utils_test: b/379936173 - Disable ffmpeg: b/378909148 This PR also fixed building content_shell_apk target on android. b/379927913
youtube · Nov 20, 2024 · 9e46da9 · 9e46da9
1 parent 4a8dced
commit 9e46da9
Show file tree

Hide file tree

Showing 10 changed files with 20 additions and 12 deletions.
diff --git a/.github/config/android-arm.json b/.github/config/android-arm.json
@@ -13,7 +13,8 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
-        "content_shell",
+        "media_unittests",
+        "content_shell_apk",
         "system_webview_apk",
         "system_webview_shell_apk",
         "cobalt:gn_all"

diff --git a/.github/config/android-arm64.json b/.github/config/android-arm64.json
@@ -13,7 +13,8 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
-        "content_shell",
+        "media_unittests",
+        "content_shell_apk",
         "system_webview_apk",
         "system_webview_shell_apk",
         "cobalt:gn_all"

diff --git a/.github/config/android-x86.json b/.github/config/android-x86.json
@@ -13,7 +13,8 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
-        "content_shell",
+        "media_unittests",
+        "content_shell_apk",
         "system_webview_apk",
         "system_webview_shell_apk",
         "cobalt:gn_all"

diff --git a/.github/config/chromium_android-arm.json b/.github/config/chromium_android-arm.json
@@ -13,7 +13,8 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
-        "content_shell",
+        "media_unittests",
+        "content_shell_apk",
         "system_webview_apk",
         "system_webview_shell_apk"
     ],

diff --git a/.github/config/chromium_android-arm64.json b/.github/config/chromium_android-arm64.json
@@ -13,7 +13,8 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
-        "content_shell",
+        "media_unittests",
+        "content_shell_apk",
         "system_webview_apk",
         "system_webview_shell_apk"
     ],

diff --git a/.github/config/chromium_android-x86.json b/.github/config/chromium_android-x86.json
@@ -13,7 +13,8 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
-        "content_shell",
+        "media_unittests",
+        "content_shell_apk",
         "system_webview_apk",
         "system_webview_shell_apk"
     ],

diff --git a/.github/config/chromium_linux.json b/.github/config/chromium_linux.json
@@ -13,6 +13,7 @@
         "gpu_unittests",
         "gin_unittests",
         "blink_unittests",
+        "media_unittests",
         "content_shell"
     ],
     "includes": [

diff --git a/media/BUILD.gn b/media/BUILD.gn
@@ -203,7 +203,10 @@ test("media_unittests") {
   ]
 
   if (is_cobalt) {
-    deps += ["//media/starboard:unit_tests"]
+    # TODO(b/379934658): add starboard_renderer_test.
+    # TODO(b/379936173): enable starboard_utils_test.
+    # TODO(b/379934533): enable bidirectional_fit_reuse_allocator_test.
+    # deps += ["//media/starboard:unit_tests"]
   }
 
   data = [

diff --git a/media/base/mock_filters.h b/media/base/mock_filters.h
@@ -140,6 +140,9 @@ class MockPipeline : public Pipeline {
   }
   MOCK_METHOD2(OnSetCdm,
                void(CdmContext* cdm_context, CdmAttachedCB& cdm_attached_cb));
+#if BUILDFLAG(USE_STARBOARD_MEDIA)
+  MOCK_METHOD0(GetSetBoundsCB, SetBoundsCB());
+#endif // BUILDFLAG(USE_STARBOARD_MEDIA)
 };
 
 class MockMediaResource : public MediaResource {

diff --git a/media/media_options.gni b/media/media_options.gni
@@ -57,11 +57,6 @@ declare_args() {
     media_use_libvpx = false
   }
 
-  # Chrobalt doesn't use ffmpeg.
-  if (is_cobalt) {
-    media_use_ffmpeg = false
-  }
-
   # Enable usage of OpenH264 within the media library. Used for software based
   # encoding of H264 content.
   media_use_openh264 = true