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

Unified Diff: content/browser/BUILD.gn

Issue 2867213004: [Mojo Video Capture] Hook up video capture service behind a feature flag (Closed)
Patch Set: Rebase to May 19th Created 3 years, 7 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
« no previous file with comments | « no previous file | content/browser/renderer_host/media/media_stream_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/BUILD.gn
diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn
index 0f85ded5f31f761f931ab846a01a33dd9c9be69e..f87748465db77dbba99ab07aeac921bde7a1a66f 100644
--- a/content/browser/BUILD.gn
+++ b/content/browser/BUILD.gn
@@ -11,12 +11,6 @@ import("//printing/features/features.gni")
import("//third_party/WebKit/public/public_features.gni")
import("//tools/ipc_fuzzer/ipc_fuzzer.gni")
-declare_args() {
- # Include code in the build that is only needed when using the video
- # capture Mojo service, which is currently experimental.
- enable_mojo_video_capture = false
-}
-
source_set("browser") {
# Only the public target should depend on this. All other targets (even
# internal content ones) should depend on the public one.
@@ -130,6 +124,8 @@ source_set("browser") {
"//services/shape_detection/public/interfaces",
"//services/ui/gpu/interfaces",
"//services/ui/public/cpp/gpu",
+ "//services/video_capture/public/cpp",
+ "//services/video_capture/public/interfaces:constants",
"//skia",
"//sql",
"//storage/browser",
@@ -1232,6 +1228,12 @@ source_set("browser") {
"renderer_host/media/renderer_audio_output_stream_factory_context.h",
"renderer_host/media/renderer_audio_output_stream_factory_context_impl.cc",
"renderer_host/media/renderer_audio_output_stream_factory_context_impl.h",
+ "renderer_host/media/service_launched_video_capture_device.cc",
+ "renderer_host/media/service_launched_video_capture_device.h",
+ "renderer_host/media/service_video_capture_device_launcher.cc",
+ "renderer_host/media/service_video_capture_device_launcher.h",
+ "renderer_host/media/service_video_capture_provider.cc",
+ "renderer_host/media/service_video_capture_provider.h",
"renderer_host/media/video_capture_controller.cc",
"renderer_host/media/video_capture_controller.h",
"renderer_host/media/video_capture_controller_event_handler.h",
@@ -1243,6 +1245,8 @@ source_set("browser") {
"renderer_host/media/video_capture_manager.cc",
"renderer_host/media/video_capture_manager.h",
"renderer_host/media/video_capture_provider.h",
+ "renderer_host/media/video_capture_provider_switcher.cc",
+ "renderer_host/media/video_capture_provider_switcher.h",
"renderer_host/native_web_keyboard_event_aura.cc",
"renderer_host/native_web_keyboard_event_mac.mm",
"renderer_host/offscreen_canvas_provider_impl.cc",
@@ -1628,22 +1632,6 @@ source_set("browser") {
]
}
- if (enable_mojo_video_capture) {
- sources += [
- "renderer_host/media/service_launched_video_capture_device.cc",
- "renderer_host/media/service_launched_video_capture_device.h",
- "renderer_host/media/service_video_capture_device_launcher.cc",
- "renderer_host/media/service_video_capture_device_launcher.h",
- "renderer_host/media/service_video_capture_provider.cc",
- "renderer_host/media/service_video_capture_provider.h",
- ]
-
- deps += [
- "//services/video_capture/public/cpp",
- "//services/video_capture/public/interfaces:constants",
- ]
- }
-
# Desktop screen capture implementations, conditionally built depending on
# the available implementations for each platform.
if (is_linux || is_mac || is_win) {
« no previous file with comments | « no previous file | content/browser/renderer_host/media/media_stream_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698