Build public cast_shell on x86 with GN.

chromecast/renderer/BUILD.gn

Index: chromecast/renderer/BUILD.gn
diff --git a/chromecast/renderer/BUILD.gn b/chromecast/renderer/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..2616e4fcf5ef0704830cc2b12ca2240b761ff148
--- /dev/null
+++ b/chromecast/renderer/BUILD.gn
@@ -0,0 +1,46 @@
+# Copyright 2015 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import("//chromecast/chromecast.gni")
+
+source_set("renderer") {
+  sources = [
+    "cast_content_renderer_client.cc",
+    "cast_content_renderer_client.h",
+    "cast_media_load_deferrer.cc",
+    "cast_media_load_deferrer.h",
+    "cast_render_process_observer.cc",
+    "cast_render_process_observer.h",
+    "key_systems_cast.cc",
+    "key_systems_cast.h",
+
+    # TODO(slan): Can these be added into "//chromecast/renderer/media"?

[gunsch, 2015/08/05 02:04:11]

I don't understand, why can't they already? ISTM like a one-way dependency of
chromecast/renderer on chromecast/renderer/media

[slan, 2015/08/05 14:34:58]

Huh. You're correct. Moved.

+    "media/capabilities_message_filter.cc",
+    "media/capabilities_message_filter.h",
+  ]
+
+  if (chromecast_branding == "public") {
+    sources += [ "cast_content_renderer_client_simple.cc" ]
+  }
+
+  public_deps = [
+    "//chromecast/renderer/media",

[gunsch, 2015/08/05 02:04:10]

Note: this and other media deps should only be here if (linux)---both this file,
chromecast/browser, and chromecast/common. This should correspond to the
"cast_shell_media" target in GYP, which will not build on Android.

As a side note, one potential benefit to the Mojo path is that it should (?)
build on Android. Our CMA pipeline actually fails at link time due to duplicate
definitions, since we have to create the same "IPC traits" classes for an
IPC-based media pipeline that Android does, and they're internal to content/ so
we can't include the declarations. :(

[slan, 2015/08/05 14:34:58]

Noted. I've added a (!is_android) conditional. I will keep an eye on this for
when we start the Android build.
Wholesale unification of code, especially of IPC, would be awesome.

+  ]
+
+  deps = [
+    "//base",
+    "//chromecast/base",
+    "//chromecast/crash",
+    "//chromecast/common/media",
+    "//chromecast/media",
+    "//components/cdm/renderer",
+    "//components/network_hints/renderer",
+    "//content/public/common",
+    "//content/public/renderer",
+    "//crypto",
+    "//ipc",
+    "//media",
+    "//third_party/widevine/cdm:version_h",
+  ]
+}