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

Unified Diff: media/media_options.gni

Issue 2289543003: IPC->mojo of audio_renderer_host (Closed)
Patch Set: It builds \o/. Created 4 years, 3 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/media_options.gni
diff --git a/media/media_options.gni b/media/media_options.gni
index 09e1cc2280e782389522e25842d89b666d109a95..67a570a0cd9ca9075193129d6ad6231916e88de6 100644
--- a/media/media_options.gni
+++ b/media/media_options.gni
@@ -77,6 +77,9 @@ declare_args() {
# Enable the TestMojoMediaClient to be used in MojoMediaApplication. This is
# for testing only and will override the default platform MojoMediaClient.
enable_test_mojo_media_client = false
+
+ # Used to enable mojo WebAudio rendering service.
+ enable_mojo_audio = true
}
# Use a second declare_args() to pick up possible overrides of enable_mojo_media
@@ -122,4 +125,23 @@ declare_args() {
mojo_media_host = "gpu"
}
}
+
+ # The process to host the mojo audio application.
+ # Valid options are:
+ # - "none": Do not use mojo audio application.
+ # - "browser": Use mojo audio application hosted in the browser process.
+ # - "gpu": Use mojo audio application hosted in the gpu process.
o1ka 2016/09/02 07:31:46 Where does this list of options come from?
+ # - "utility": Use mojo audio application hosted in the utility process.
+ mojo_audio_host = "none"
+
+ if (enable_mojo_audio) {
+ mojo_audio_host = "browser"
+ }
+}
+
+if (enable_mojo_audio) {
+ assert(mojo_audio_host == "browser",
+ "Mojo audio not supported outside browser process.")
+} else {
+ assert(mojo_audio_host == "none", "Mojo audio is not enabled.")
}

Powered by Google App Engine
This is Rietveld 408576698