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

Unified Diff: media/media_options.gni

Issue 2115603002: Use FFmpegVideoDecoder from a UtilityProcess (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix build error on win Created 4 years 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 61464d48fdee0bb4292870187f939158c2144862..5a34e81fc86c6debc00598d985303e7a79c8ae4b 100644
--- a/media/media_options.gni
+++ b/media/media_options.gni
@@ -74,7 +74,7 @@ declare_args() {
# |mojo_media_services|). When enabled, selected mojo paths will be enabled in
# the media pipeline and corresponding services will hosted in the selected
# remote process (e.g. "utility" process, see |mojo_media_host|).
- enable_mojo_media = is_android || is_chromecast
+ enable_mojo_media = is_android || is_chromecast || is_desktop_linux
# Enable the TestMojoMediaClient to be used in mojo MediaService. This is for
# testing only and will override the default platform MojoMediaClient, if any.
@@ -119,6 +119,9 @@ declare_args() {
"renderer",
]
mojo_media_host = "browser"
+ } else if (is_desktop_linux) {
sandersd (OOO until July 31) 2016/12/15 00:38:02 |is_desktop_linux| seems to be defined as |current
+ mojo_media_services = [ "video_decoder" ]
+ mojo_media_host = "utility"
} else {
mojo_media_services = [ "video_decoder" ]
sandersd (OOO until July 31) 2016/12/15 00:38:02 Do you plan to expand past desktop linux eventuall
mojo_media_host = "gpu"

Powered by Google App Engine
This is Rietveld 408576698