media: Make media mojo services a component

content/gpu/BUILD.gn

Index: content/gpu/BUILD.gn
diff --git a/content/gpu/BUILD.gn b/content/gpu/BUILD.gn
index 63d8adf13dea5b688b294cc35f35ac918db2ba66..ba1a4a4c54591817dece098b116df4642be2fb70 100644
--- a/content/gpu/BUILD.gn
+++ b/content/gpu/BUILD.gn
@@ -57,7 +57,6 @@ source_set("gpu_sources") {
     "//ipc",
     "//media/gpu",
     "//media/gpu/ipc/service",
-    "//media/mojo/services:application_factory",

[xhwang, 2016/05/12 06:32:25]

+brettw: This was added in https://codereview.chromium.org/1833193002, even
though we have the conditional deps below at line 69. And in the source code we
have a conditional include.

Are we preferring unconditional deps to nogncheck now?

     "//services/shell/public/interfaces",
     "//skia",
     "//ui/events/ipc",
@@ -66,7 +65,7 @@ source_set("gpu_sources") {
   ]
 
   if (mojo_media_host == "gpu") {
-    deps += [ "//media/mojo/services:application_factory" ]
+    deps += [ "//media/mojo/services" ]

[ddorwin, 2016/05/11 23:02:27]

This was previously always included. Was that incorrect?

[xhwang, 2016/05/16 05:41:20]

We should either only always include, or only conditional include. I think
conditional include makes more sense.

     if (is_android) {
       deps += [ "//media/base/android" ]
     }