media: Support MojoRendererService in the browser process.

media/mojo/services/BUILD.gn

Index: media/mojo/services/BUILD.gn
diff --git a/media/mojo/services/BUILD.gn b/media/mojo/services/BUILD.gn
index c9e7a262c99e572b324a1709d30e0931185f9583..1f9e625912a1d8141a877ff676ff4397f93c1f12 100644
--- a/media/mojo/services/BUILD.gn
+++ b/media/mojo/services/BUILD.gn
@@ -2,6 +2,7 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import("//media/media_options.gni")
 import("//mojo/public/mojo_application.gni")
 
 # Target naming conventions:
@@ -11,6 +12,13 @@ import("//mojo/public/mojo_application.gni")
 # - unittests: Unit tests for a particular class/file.
 # - test: Tests for a particular app, e.g. media.
 
+config("media_mojo_dependent_config") {

[jamesr, 2014/12/18 17:49:54]

nit: i would name this after what it does, not how it's used.

[xhwang, 2014/12/18 21:22:37]

Done.

+  defines = []

[jamesr, 2014/12/18 17:49:54]

nit: you don't have to have this, you can instead just say "defines = [ ... ] "
inside the if block

[xhwang, 2014/12/18 21:22:38]

Done.

+  if (enable_media_mojo_renderer) {
+    defines += [ "ENABLE_MEDIA_MOJO_RENDERER" ]
+  }
+}
+
 source_set("converters") {
   sources = [
     "media_type_converters.cc",
@@ -41,6 +49,8 @@ source_set("renderer_proxy") {
     "mojo_renderer_impl.h",
   ]
 
+  all_dependent_configs = [ ":media_mojo_dependent_config" ]

[jamesr, 2014/12/18 17:49:54]

all_dependent_configs is bad - you'll set this define in lots of targets that
have nothing to do with this.  use public_configs and and, where necessary,
forward_dependent_configs_from to forward this as needed

[DaleCurtis, 2014/12/18 19:25:11]

Can you elaborate on how this will pollute non-dependent targets?

[xhwang, 2014/12/18 21:22:38]

Good point! Thanks!

+
   deps = [
     ":converters",
     "//base",
@@ -49,8 +59,6 @@ source_set("renderer_proxy") {
     "//mojo/common",
     "//mojo/environment:chromium",
     "//mojo/public/c/system:for_component",
-    "//mojo/public/cpp/application",
-    "//mojo/public/interfaces/application",
   ]
 }
 
@@ -68,6 +76,8 @@ source_set("renderer_service") {
     "renderer_config_default.cc",
   ]
 
+  all_dependent_configs = [ ":media_mojo_dependent_config" ]

[jamesr, 2014/12/18 17:49:54]

ditto

[xhwang, 2014/12/18 21:22:38]

Done.

+
   deps = [
     ":converters",
     "//base",